On Mon, Jul 01, 2019 at 01:13:43PM +0200, Charlene Wendling wrote: > Ping. I've got no news from upstream [0], but i still think it's better > to deliver this than the current one, that is broken at runtime. > > I'm attaching a diff with the PERMIT_PACKAGE change done.
OK afresh1@ > Charlène. > > [0] https://rt.cpan.org/Public/Bug/Display.html?id=129728 > > On Sat, 1 Jun 2019 20:08:53 +0200 > Charlene Wendling wrote: > > > Hi, > > > > On Wed, 29 May 2019 19:30:52 -0700 > > Andrew Hewus Fresh wrote: > > > > > On Tue, May 21, 2019 at 08:12:30AM +0200, Charlene Wendling wrote: > > > > > > [...] > > > > > > This one needs: > > > RUN_DEPENDS+= www/p5-LWP-Protocol-https > > > > Yes, it makes sense indeed. > > > > > and a comment that tests need the network to run, similar to > > > p5-WWW-Tumbler. > > > > > > It still doesn't quite seem to work right when I use: > > > AutoSearch --engine Ebay -n BSD -s bsd bsd > > > > > > it does work though, check out: > > > https://www.ebay.com/itm/COMDEX-1999-special-1-4M-preview-release-of-the-NetBSD-operating-system-CD-ROM/333212907390?hash=item4d9509777e:g:GQcAAOSwX1xc2Wjm > > > > > > But lots of warnings for: > > > Use of uninitialized value in numeric lt (<) > > > at /usr/local/libdata/perl5/site_pe rl/WWW/Search/Ebay.pm line 672. > > > > > > This patch seems to fix it, but I don't know if it's the right fix > > > or if there's some other extenuating reason that the end_date is > > > sometimes undefined. I do see a few wide characters in the output > > > so I wonder if it's just not very unicode safe and stuff is going > > > wrong due to that. > > > > > > > > [...] > > > > I'm sending an updated diff that deals with these issues. There seems > > to be a problem in the HTML parsing code. I thought i found out why, > > but it appears that's not it. Debug logs seem interesting for > > upstream though, so i'll report there - meanwhile your fix and that > > update are better than what we have currently :) > > > > Charlène. > > > > -- > > > andrew - http://afresh1.com > > > > > > Hey, I think I see a barn up ahead. > > > -- The American Astronaut > > > > > > Index: Makefile > =================================================================== > RCS file: /cvs/ports/www/p5-WWW-Search-Ebay/Makefile,v > retrieving revision 1.14 > diff -u -p -u -p -r1.14 Makefile > --- Makefile 6 Aug 2017 20:12:57 -0000 1.14 > +++ Makefile 1 Jul 2019 11:12:14 -0000 > @@ -2,22 +2,28 @@ > > COMMENT= backend for searching www.ebay.com > > -DISTNAME = WWW-Search-Ebay-3.042 > -REVISION = 1 > +DISTNAME= WWW-Search-Ebay-3.052 > CATEGORIES= www > > -# perl > -PERMIT_PACKAGE_CDROM= Yes > +# Perl > +PERMIT_PACKAGE= Yes > > MODULES= cpan > PKG_ARCH= * > > -BUILD_DEPENDS = devel/p5-Module-Install-AuthorTests > -RUN_DEPENDS= www/p5-WWW-Search > +BUILD_DEPENDS= devel/p5-Module-Install-AuthorTests > + > +RUN_DEPENDS= converters/p5-DateManip \ > + www/p5-HTML-Tree \ > + www/p5-LWP-Protocol-https \ > + www/p5-WWW-Search>=2.517 \ > + www/p5-libwww > + > +# Tests need network access > TEST_DEPENDS= devel/p5-IO-Capture > > MAKE_ENV+= TEST_POD="Yes" > > -CONFIGURE_STYLE = modinst > +CONFIGURE_STYLE= modinst > > .include <bsd.port.mk> > Index: distinfo > =================================================================== > RCS file: /cvs/ports/www/p5-WWW-Search-Ebay/distinfo,v > retrieving revision 1.6 > diff -u -p -u -p -r1.6 distinfo > --- distinfo 19 Sep 2015 10:05:48 -0000 1.6 > +++ distinfo 1 Jul 2019 11:12:14 -0000 > @@ -1,2 +1,2 @@ > -SHA256 (WWW-Search-Ebay-3.042.tar.gz) = > IJnzUNMmnbHXrgb8Tbxbjb5QJ0wlu7dzCnFSYVPER5U= > -SIZE (WWW-Search-Ebay-3.042.tar.gz) = 54585 > +SHA256 (WWW-Search-Ebay-3.052.tar.gz) = > osgshTeJPvhLfwLOQoGN+TW4F7cL0EBrgoE6iA6R0Gg= > +SIZE (WWW-Search-Ebay-3.052.tar.gz) = 54632 > Index: patches/patch-lib_WWW_Search_Ebay_pm > =================================================================== > RCS file: patches/patch-lib_WWW_Search_Ebay_pm > diff -N patches/patch-lib_WWW_Search_Ebay_pm > --- /dev/null 1 Jan 1970 00:00:00 -0000 > +++ patches/patch-lib_WWW_Search_Ebay_pm 1 Jul 2019 11:12:14 -0000 > @@ -0,0 +1,27 @@ > +$OpenBSD$ > + > +Fix for: > +Use of uninitialized value in numeric lt (<) at > +/usr/local/libdata/perl5/site_perl/WWW/Search/Ebay.pm line 672. > + > +Index: lib/WWW/Search/Ebay.pm > +--- lib/WWW/Search/Ebay.pm.orig > ++++ lib/WWW/Search/Ebay.pm > +@@ -669,7 +669,7 @@ sub result_as_HTML > + my $dateNow = ParseDate('now'); > + print STDERR " DDD compare end_date ==$dateEnd==\n" if (DEBUG_DATES || (1 > < $self->{_debug})); > + print STDERR " DDD compare date_now ==$dateNow==\n" if (DEBUG_DATES || (1 > < $self->{_debug})); > +- if (Date_Cmp($dateEnd, $dateNow) < 0) > ++ if ((Date_Cmp($dateEnd, $dateNow) || 0) < 0) > + { > + $sEndedColor = 'red'; > + $sEndedWord = 'ended'; > +@@ -1040,7 +1040,7 @@ sub _parse_tree > + && > + (0 < $iBids) # Item got any bids > + && > +- (Date_Cmp($enddate, 'now') < 0) # Item is ended > ++ ((Date_Cmp($enddate, 'now') || 0) < 0) # Item is ended > + ) > + { > + # Item must have been sold!?! > -- andrew - http://afresh1.com I think I understand, but my stubborn brain refuses to admit it until I beat it into submission by proof upon proof. -- Michael Shiloh <[email protected]>
