Re: Please review wip/tcl-tDOM

Moritz Wilhelmy <[email protected]>
Newsgroups gmane.os.netbsd.devel.pkgsrc.wip.general
Message-ID <[email protected]>
On Thu, Dec 29, 2011 at 16:10:06 +0000, [email protected] wrote:
> Hrmm. Don't know what happened there with the cutoff mail.
> 
> re buildlink3: 
> 
> =========================
> kamloops# make 
> ERROR: This package has set PKG_FAIL_REASON:
> ERROR: Out-dated buildlink3.mk detected, please update
> *** Error code 1
> 
> Stop.
> make: stopped in /usr/pkgsrc/wip/tcl-tDOM
> =========================

And that's in /usr/pkgsrc/wip/tcl-tDOM? I really don't see one here...
I'm fairly new to pkgsrc and unsure whether it should provide a
buildlink3.mk or not.

> I think one of my questions from the mangled prev. email was if you're
> actually building and using tdom, and in what env (platform, version,
> Tcl version, etc).

NetBSD/i386 5.99.56, Tcl 8.4.18, it loads and I am able to parse XML
input:

% package require tdom                                   
0.8.2
% set d [dom parse {<xml foo="boo"><doo>moo</doo></xml>}]
domDoc0xbb8bd180
% [[$d firstChild] firstChild] text
moo

I should also run the unit-tests, but I had no reason to believe that it
doesn't work regardless.

> Also note, tdom is up to 0.8.3 now. I've also got a patch for tdom moving
> foward to Tcl 8.6, and some considerations for build options. Will try
> to post later today.

Must have overlooked that. Feel free to commit your fixes directly into
the subdirectory. Since that's a minor upgrade (AFAIK, only the in-tree
Expat was updated because a CVE was fixed), it shouldn't be hard to just
bump the version number.


Best regards,

Moritz

------------------------------------------------------------------------------
Ridiculously easy VDI. With Citrix VDI-in-a-Box, you don't need a complex
infrastructure or vast IT resources to deliver seamless, secure access to
virtual desktops. With this all-in-one solution, easily deploy virtual 
desktops for less than the cost of PCs and save 60% on VDI infrastructure 
costs. Try it free! http://p.sf.net/sfu/Citrix-VDIinabox
lmpx.com only provides a reader for public news (NNTP) servers. It is not affiliated with the servers or forums shown here and is not responsible for the content of articles, which is written by their respective authors.