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. 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!?!
