Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 95166 - xmltv-0.5.43 out - new ebuild using USE
Summary: xmltv-0.5.43 out - new ebuild using USE
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All All
: High enhancement (vote)
Assignee: Television related Applications in Gentoo's Portage
URL:
Whiteboard:
Keywords:
: 91568 100926 113782 114933 126198 (view as bug list)
Depends on: 135775 140447
Blocks: 94101
  Show dependency tree
 
Reported: 2005-06-05 18:50 UTC by Kurt Hindenburg
Modified: 2007-03-21 21:31 UTC (History)
20 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments
ebuild for 0.5.40 using USE (xmltv-0.5.40.ebuild,4.92 KB, text/plain)
2005-06-05 18:51 UTC, Kurt Hindenburg
Details
A more complete ebuild (xmltv-0.5.40.ebuild,4.89 KB, text/plain)
2005-06-07 09:59 UTC, Kurt Hindenburg
Details
Try this again. (xmltv-0.5.40.ebuild,4.92 KB, text/plain)
2005-06-07 11:01 UTC, Kurt Hindenburg
Details
xmltv-0.5.41 ebuild working ALSO for foreign grabbers (the other one won't) (xmltv-0.5.41.ebuild,5.39 KB, text/plain)
2005-11-28 11:27 UTC, Matteo Azzali (RETIRED)
Details
working download also (xmltv-0.5.41.ebuild,5.39 KB, text/plain)
2005-12-01 11:59 UTC, Matteo Azzali (RETIRED)
Details
working download also (xmltv-0.5.41.ebuild,5.39 KB, text/plain)
2005-12-01 11:59 UTC, Matteo Azzali (RETIRED)
Details
MethodMaker-1dep.for na_dd use flag fix. (0.5.42-r1) (xmltv-0.5.42-r1.ebuild,5.44 KB, text/plain)
2006-01-14 09:41 UTC, Matteo Azzali (RETIRED)
Details
xmltv-0.5.42-r2.ebuild (check notes below) (xmltv-0.5.42-r2.ebuild,5.52 KB, text/plain)
2006-03-01 05:11 UTC, Matteo Azzali (RETIRED)
Details
xmltv-0.5.42-r2.ebuild (xmltv-0.5.42-r2.ebuild,5.59 KB, text/plain)
2006-04-23 04:58 UTC, Matteo Azzali (RETIRED)
Details
xmltv-0.5.43.ebuild (xmltv-0.5.43.ebuild,5.72 KB, text/plain)
2006-04-27 05:25 UTC, Matteo Azzali (RETIRED)
Details
xmltv-0.5.43-r1.ebuild (xmltv-0.5.43-r1.ebuild,5.65 KB, text/plain)
2006-05-28 03:15 UTC, Leszek
Details
tv_grab_de_tvtoday-0.5.43.diff (tv_grab_de_tvtoday-0.5.43.diff,4.92 KB, patch)
2006-05-30 08:26 UTC, Horst Schirmeier
Details | Diff
xmltv-0.5.43-r1.ebuild (xmltv-0.5.43-r1.ebuild,5.62 KB, text/plain)
2006-05-30 08:30 UTC, Horst Schirmeier
Details
xmltv-0.5.43-r2.ebuild (alternative, old XMLTV_OPTS version) (xmltv-0.5.43-r2.ebuild,8.79 KB, text/plain)
2006-05-30 10:00 UTC, Matteo Azzali (RETIRED)
Details
tv_grab_de_tvtoday-0.5.43.diff without problematic first chunk (tv_grab_de_tvtoday-0.5.43.diff,4.53 KB, patch)
2006-06-01 06:29 UTC, Horst Schirmeier
Details | Diff
xmltv-0.5.43-r4.ebuild (xmltv-0.5.43-r4.ebuild,8.32 KB, text/plain)
2006-06-01 19:42 UTC, Matteo Azzali (RETIRED)
Details
xmltv-0.5.43-r4.ebuild (xmltv-0.5.43-r4.ebuild,8.47 KB, text/plain)
2006-06-02 03:30 UTC, Matteo Azzali (RETIRED)
Details
Move make to src_compile and use perl-module functions (xmltv-0.5.43-r4.ebuild.patch,651 bytes, patch)
2006-06-10 14:45 UTC, Anders Hellgren
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Kurt Hindenburg 2005-06-05 18:50:57 UTC
0.5.40 is out.  I've changed the ebuild to use IUSE instead of XMLOPTS

See:http://bugs.gentoo.org/show_bug.cgi?id=91568


Reproducible: Always
Steps to Reproduce:
1.
2.
3.
Comment 1 Kurt Hindenburg 2005-06-05 18:51:33 UTC
Created attachment 60698 [details]
ebuild for 0.5.40 using USE
Comment 2 Kurt Hindenburg 2005-06-07 09:59:01 UTC
Created attachment 60781 [details]
A more complete ebuild

I don't use tv_pick_cgi, but this should work.
Comment 3 Kurt Hindenburg 2005-06-07 11:01:04 UTC
Created attachment 60785 [details]
Try this again.
Comment 4 Horst Schirmeier 2005-09-17 05:01:11 UTC
This ebuild with USE=de_tvtoday works quite well here, together with the
grab_tv_de_tvtoday 0.5.40 patch from bug 100926, comment 8.

Please consider including in Portage, as the local USE flags solution is far
cleaner than the $XMLTV_OPTS environment variable.
Comment 5 Oscar Carlsson 2005-10-19 07:06:50 UTC
tv_grab_se_swedb needs dev-perl/XML-LibXML and dev-perl/IO-stringy as well.
Comment 6 Matteo Azzali (RETIRED) gentoo-dev 2005-11-06 12:27:34 UTC
Does not works. it doesn't get the uncommon flags, eg: it flag, es flag are 
ignored.....
Comment 7 Matteo Azzali (RETIRED) gentoo-dev 2005-11-07 09:28:50 UTC
The issue for foreign tv_grab not emerged seems caused by the new beahviour at 
compile time: "perl Makefile.PL ;make ;make install" just does make for na and uk 
grabbers, foreign users need to do "perl Makefile.PL ;make ;make test ;make
install" so for a general ebuild "MAKE TEST" is not optional.

The issue start when you try it and discover that "make test" wants to change 
permissions for some manual (man) file, and sandbox complains about it.....
Comment 8 Matteo Azzali (RETIRED) gentoo-dev 2005-11-25 04:48:32 UTC
0.5.41 out....
Comment 9 Jakub Moc (RETIRED) gentoo-dev 2005-11-28 08:16:02 UTC
*** Bug 113782 has been marked as a duplicate of this bug. ***
Comment 10 Matteo Azzali (RETIRED) gentoo-dev 2005-11-28 11:27:40 UTC
Created attachment 73751 [details]
xmltv-0.5.41 ebuild working ALSO for foreign grabbers (the other one won't)

ehm, wasn't a duplicate bug, this ebuild IS DIFFERENT, and works for foreign
grabbers too. It solves the issues I described above (chmod and sandbox..)
Comment 11 Matteo Azzali (RETIRED) gentoo-dev 2005-11-29 03:04:21 UTC
just a note: au , ch , is , ee , re could not be 100% accurate in dependancies.
if one (or more) of these aren't working, a line should be added with html-tree
dependancy near other similar dependancy lines , ex:

au? ( dev-perl/HTML-Tree )

That's since documentation isn't 100% precise about that....

Other than this, the ebuild has been tested and working , both for native and 
foreign flags (someone talked about "issues upgrading", I'm doin further 
investigations )
Comment 12 Matteo Azzali (RETIRED) gentoo-dev 2005-11-29 12:35:01 UTC
"issues upgrading" were only complaints about the speed (or better, the slowness)
of the xmltv grabber, (I don't know if this is related just to this version,
and if this is a wanted feature to not overload servers or just a bug),
anything else is fine (except dependancy of au , ch , is , ee , re USE FLAGS,
which may depend on the package dev-perl/HTML-Tree, but we'll not know
if someone from these country will not test).
Ebuild is ready to be added in portage, IMHO.
Comment 13 Matteo Azzali (RETIRED) gentoo-dev 2005-12-01 11:59:43 UTC
Created attachment 73901 [details]
working download also

Modified SRC_URI to download it fine.
Comment 14 Matteo Azzali (RETIRED) gentoo-dev 2005-12-01 11:59:53 UTC
Created attachment 73902 [details]
working download also

Modified SRC_URI to download it fine.
Comment 15 Matteo Azzali (RETIRED) gentoo-dev 2005-12-04 07:44:28 UTC
actually tested (and working)by german user 
http://forums.gentoo.org/viewtopic-p-2914415.html#2914415

and some italian users, 
http://forums.gentoo.org/viewtopic-p-2913294.html#2913294

please consider to test and add in portage this version bump.
(I can add ewarnings in the ebuild if needed, for the change to the local use 
flags...)
Comment 16 Matteo Azzali (RETIRED) gentoo-dev 2005-12-09 12:46:55 UTC
P.S.: this ebuild IS working with MethodMaker 2.07-r2 which is in portage,at
least with x86 / AMD64 , and this mean security improvement (not using anymore
a stale package as dependancy....)
Comment 17 Kurt Hindenburg 2005-12-24 19:57:06 UTC
Why did you re-add es_digital?  It does not work... from the Makefile.PL

#
# tv_grab_es_digital no longer functioning due to site changes
#
#     { name => 'tv_grab_es_digital',
#       blurb => 'Grabber for Spain - Digital Satellite (D+)',
#       exes => [ 'grab/es_digital/tv_grab_es_digital' ],
#       prereqs => { 'HTML::TreeBuilder' => 0 } },


What add this?
PREFIX="/usr"
Comment 18 Matteo Azzali (RETIRED) gentoo-dev 2005-12-25 10:51:25 UTC
Ehm, all the lines containing es_digital were marked out (that's the meaning of 
a # at the start of the line) just cause I thinked they could be handful there
when updating the ebuild to a new version (just in case es_digital will return to 
work)...
es_digital isn't an use flag and all their lines are commented out....
Comment 19 Matteo Azzali (RETIRED) gentoo-dev 2005-12-25 10:54:06 UTC
PREFIX=/usr was needed since without it some of the resulting files for foreign
languages would have been putted inside /share/xmltv instead of /usr/share/xmltv
during "make test" phase.
Comment 20 Matteo Azzali (RETIRED) gentoo-dev 2005-12-25 10:56:05 UTC
Oh, I forgot: using PREFIX=/usr is equivalent to do: ./configure --prefix=/usr
Comment 21 Kurt Hindenburg 2005-12-25 12:08:32 UTC
O.K.  I'm using na_dd.

1. dev-perl/Class-MethodMaker-1.11 works fine for me.  Why do you require >=2 version (which is masked)?

2. The tests takes quite a while...

Now all we need is to find a Gentoo maintainer... I've offered but no takers.
Comment 22 Doug Goldstein (RETIRED) gentoo-dev 2005-12-25 22:27:28 UTC
*** Bug 91568 has been marked as a duplicate of this bug. ***
Comment 23 Doug Goldstein (RETIRED) gentoo-dev 2005-12-25 22:29:15 UTC
*** Bug 100926 has been marked as a duplicate of this bug. ***
Comment 24 Doug Goldstein (RETIRED) gentoo-dev 2005-12-25 22:36:41 UTC
*** Bug 114933 has been marked as a duplicate of this bug. ***
Comment 25 Matteo Azzali (RETIRED) gentoo-dev 2005-12-26 02:36:44 UTC
1) MethodMaker 2.07-r2 isn't masked, just unstable (as this xmltv would be).
Is also working (at least for Italy and Germany) and is "supported", while
MethodMaker 1.11 is really old and not mantained (supported) anymore.
I suspect the old ebuild were using 1.11 cause 2.0 was too young and didn't 
worked, if there are better reasons to use this obsolete version please tell me.

2) I'm sorry if the tests takes a while, but without them seems that some foreign
(other thank unk and na) grabbers won't work, so removing them renders xmltv 
useless for who's not in uk or na. (and I think the ebuild should be 
general-purpose, not only for na and uk users, correct me if I'm wrong)
Comment 26 Kurt Hindenburg 2006-01-10 10:39:19 UTC
AFAIK the .41 ebuild works O.K. with the .42 release.

Comment 27 Kurt Hindenburg 2006-01-14 06:56:09 UTC
Note: Using ~x86 Class-MethodMaker-2.0+ gives me this error when I try to use tv_grab_na_dd:

Can't locate object method "new" via package "Term::ProgressBar" at /usr/lib/perl5/site_perl/5.8.6/XMLTV/ProgressBar/Term.pm line 15.

I tried re-emerging xmltv and ProgressBar (2.06/09), but could not get rid of the error above.

I went back to Class-MethodMaker-1.11
Comment 28 Matteo Azzali (RETIRED) gentoo-dev 2006-01-14 09:41:01 UTC
Created attachment 77087 [details]
MethodMaker-1dep.for na_dd use flag fix. (0.5.42-r1)

fixed the MethodMaker >2 dependancy for na_dd use flag:
If you have na_dd use flag the dependancy will be for the obsolete version,
if you don't have will be for the mantained version.
Comment 29 fab6 2006-01-27 10:02:43 UTC
tv_grab_fr --configure doesn't work :

tom@pim ~ $ tv_grab_fr --configure
using config filename /home/tom/.xmltv/tv_grab_fr.conf
A nonempty configuration file /home/tom/.xmltv/tv_grab_fr.conf
already exists.  There is currently no support for altering an
existing configuration: you have to reconfigure from scratch.

Do you wish to overwrite the old configuration? [yes,no (default=no)] yes
Get channels type : CABLE/SAT? [yes,no,all,none (default=yes)] no
Get channels type : CANAL SAT? [yes,no,all,none (default=yes)] no
Get channels type : ETRANGERES? [yes,no,all,none (default=yes)] no
Get channels type : FREEBOX? [yes,no,all,none (default=yes)] no
Get channels type : HERTZIENNE? [yes,no,all,none (default=yes)] yes
Get channels type : TNT? [yes,no,all,none (default=yes)] no
Get channels type : TPS? [yes,no,all,none (default=yes)] no
getting channel lists: No channels could be found at /usr/bin/tv_grab_fr line 289, <STDIN> line 8.


Is there a fix anywhere ?
Comment 30 Matteo Azzali (RETIRED) gentoo-dev 2006-01-28 03:37:30 UTC
Sorry, I can't find any fix for tv_grab_fr, but at least I can give you 2 links
to check with other users:
http://www.lahiette.com/biboobox/
and
http://sourceforge.net/tracker/?group_id=39046&atid=424135

It would be interesting also to know if 0.5.39 was/is working at present time,
to know if this can be a bug that blocks the commission in portage.
Comment 31 Kurt Hindenburg 2006-01-28 13:11:13 UTC
Looks like pt deps need adjusting:

These dependencies are missing for tv_grab_pt:
Unicode::UTF8simple

This dep doesn't appear to be in portage.  I'm not sure if pt will work without it...

http://search.cpan.org/~gus/Unicode-UTF8simple-1.06/
Comment 32 Matteo Azzali (RETIRED) gentoo-dev 2006-01-28 16:22:32 UTC
Sorry I have no time for creating an ebuild for Unicode::UTF8simple,
if some tv_grab_pt user create it, I'll add to ebuild,elsewhere tv_grab_pt
will be unsupported or buggy as it was in previous releases.
Sorry again but new xmltv ebuilds are here from june 2005 and I don't want
to add untested/unstable features until something will not be inserted in portage
(even masked).
Comment 33 Matteo Azzali (RETIRED) gentoo-dev 2006-03-01 05:11:40 UTC
Created attachment 81033 [details]
xmltv-0.5.42-r2.ebuild  (check notes below)

Added 0.5.42-r2 : should be compatible with perl-5.8.8 , remember you
have to do before "perl-cleaner reallyall" if you have update perl (yes, even with emerge). 
Tested working here with perl 5.8.8 and the flag "-build".
Comment 34 Jakub Moc (RETIRED) gentoo-dev 2006-03-14 11:42:57 UTC
*** Bug 126198 has been marked as a duplicate of this bug. ***
Comment 35 Horst Schirmeier 2006-03-17 16:30:50 UTC
Matteo's latest ebuild (comment #33) works fine for me with USE=de_tvtoday.
Comment 36 Kurt Hindenburg 2006-03-21 11:59:18 UTC
Add ~sparc; works for na_dd
Comment 37 Ville Mällinen 2006-04-01 09:33:07 UTC
(In reply to comment #33)
> Created an attachment (id=81033) [edit]
> xmltv-0.5.42-r2.ebuild  (check notes below)
> 
> Added 0.5.42-r2 : should be compatible with perl-5.8.8 , remember you
> have to do before "perl-cleaner reallyall" if you have update perl (yes, even
> with emerge). 
> Tested working here with perl 5.8.8 and the flag "-build".
> 
Tested on amd64 using USE=fi.The ebuild works. The grabber doesnt. Maintainer of the tv_grab_fi is resigning and the channel data source, www.katso.fi has gone through changes.

As a side note when upgrading to perl-5.8.8 and then running perl-cleaner really all downgraded perl back to v 5.7.something. Anyway after upgrading perl back to v 5.8.8 I got the ebuild finally build correctly.

Whole process left my xmltv kind of 'broken'. No matter I can use DVB-EIT data to get channel data.
Comment 38 Matteo Azzali (RETIRED) gentoo-dev 2006-04-15 06:51:04 UTC
A request to any developer that may read this accidentally: 
even 0.5.42 is slowly becoming outdated, and in portage we are stopped to 0.5.39.

There's nothing we can do to accelerate the submission in portage? 
I mean, if you want me to restore the old XMLTV_OPTS instead of USE, please tell.
Also, if this is the case, would be nice the option to use IUSE_XMLTV_OPTS like
it's done with xorg-x11 IUSE_INPUT_DEVICES and IUSE_VIDEO_CARDS , if I modify
the ebuild in that way, will be committed or there are other issues causing this
xmltv stop ?
Comment 39 Matteo Azzali (RETIRED) gentoo-dev 2006-04-23 04:58:53 UTC
Created attachment 85269 [details]
xmltv-0.5.42-r2.ebuild

just changes the dependancy memoize and storable, they're no more needed
if perl is newer than a certain release. 
This "way" was adopted by other ebuilds since some time, sorry for not having noticed.
Comment 40 Matteo Azzali (RETIRED) gentoo-dev 2006-04-27 05:12:01 UTC
Little new, located UT8Simple package needed for pt users at http://search.cpan.org/~gus/Unicode-UTF8simple-1.06/ .
If anyone is so kind to make an ebuild I'll add to deps for pt users.

Also 0.5.43 is released, with new grabbers for fr,sw,bugfixes etc.etc.
See complete changelog at http://sourceforge.net/project/shownotes.php?release_id=412239&group_id=39046
(Bottom of the page).
An updated ebuild is coming.
Comment 41 Matteo Azzali (RETIRED) gentoo-dev 2006-04-27 05:23:20 UTC
Other news, maybe bad this time: no track of australian or sweden1 (sweden2 still 
present) grabbers in this release. Were they working? My supposition is that they
were broken, if anyone wants to confirm/negate....
Comment 42 Matteo Azzali (RETIRED) gentoo-dev 2006-04-27 05:25:43 UTC
Created attachment 85602 [details]
xmltv-0.5.43.ebuild
Comment 43 Kurt Hindenburg 2006-04-28 08:44:10 UTC
Thanks for the new ebuild; I'm not sure na_dd still requires 
na_dd? ( <dev-perl/Class-MethodMaker-2 )

I'll have to test this out this weekend.
Comment 44 Leszek 2006-05-28 03:10:51 UTC
emerge xmltv -va

These are the packages that would be merged, in order:

Calculating dependencies \
!!! All ebuilds that could satisfy "<perl-core/ExtUtils-MakeMaker-6.30" have been masked.
!!! One of the following masked packages is required to complete your request:
- perl-core/ExtUtils-MakeMaker-6.21-r1 (masked by: package.mask)
# Michael Cummings <mcummings@gentoo.org> (27 May 2006)
# Masked prior to removal

- perl-core/ExtUtils-MakeMaker-6.20-r1 (masked by: package.mask)

For more information, see MASKED PACKAGES section in the emerge man page or
refer to the Gentoo Handbook.
(dependency required by "media-tv/xmltv-0.5.43" [ebuild])
Comment 45 Leszek 2006-05-28 03:15:55 UTC
Created attachment 87711 [details]
xmltv-0.5.43-r1.ebuild

i don't know if it is enough but i deleted the ExtUtils-MakeMaker dependencies
and unmerged them.
it seems to work.
Comment 46 Matteo Azzali (RETIRED) gentoo-dev 2006-05-28 07:19:00 UTC
Ok, old ebuild marked obsolete while testing.....
Comment 47 Horst Schirmeier 2006-05-30 08:17:37 UTC
0.5.43-r1 and earlier versions' tv_grab_de_tvtoday fail due to recent tvtoday.de changes:

using config filename /home/bigfoot/.xmltv/tv_grab_de_tvtoday.conf
getting list of channels: ##################################################
grabbing: cannot find date on requested page at /usr/bin/tv_grab_de_tvtoday line 509.
-: 
no element found at line 4, column 200, byte 278 at /usr/lib/perl5/vendor_perl/5.8.8/i686-linux/XML/Parser.pm line 187

Recent xmltv CVS changes in xmltv/grab/de_tvtoday/tv_grab_de_tvtoday.in (release 0.5.43 has v1.42, current CVS is v1.45) correct this problem.
Comment 48 Horst Schirmeier 2006-05-30 08:26:12 UTC
Created attachment 87868 [details, diff]
tv_grab_de_tvtoday-0.5.43.diff

Patch against xmltv-0.5.43 to get tv_grab_de_tvtoday working again (taken from upstream CVS: cvs diff -puN -r1.42 -r1.45 grab/de_tvtoday/tv_grab_de_tvtoday.in).
Comment 49 Horst Schirmeier 2006-05-30 08:30:12 UTC
Created attachment 87871 [details]
xmltv-0.5.43-r1.ebuild

revised xmltv-0.5.43-r1 ebuild:

* applies files/tv_grab_de_tvtoday-0.5.43.diff (see above)
* lots of whitespace fixes to make repoman happy (no complaints anymore besides lots of IUSE.invalid due to newly introduced language use flags)
Comment 50 Matteo Azzali (RETIRED) gentoo-dev 2006-05-30 10:00:07 UTC
Created attachment 87879 [details]
xmltv-0.5.43-r2.ebuild (alternative, old XMLTV_OPTS version)

Update not needed, but this version is more likely to get committed since 
repoman doesn't complains at all. beside XMLTV_OPTS is the same as 0.5.43-r1.

old ebuild (the same as ebuilds in portage, XMLTV_OPTS need to be defined,
uk_bleb  became tv_grab_bleb ,
tv_check and tv_pick_cgi remains the same,
for all others add tv_grab_ in front,
example: put in my /etc/make.conf :
XMLTV_OPTS="tv_grab_it tv_check" )
Comment 51 Horst Schirmeier 2006-05-30 10:28:52 UTC
xmltv-0.5.43-r2.ebuild (attachment #87879 [details]) works fine for me, tv_grab_de_tvtoday tested.

Please consider portage tree inclusion. Current testing (!) xmltv is 0.5.39, which is >14 months old.
Comment 52 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 03:08:31 UTC
already committed (also solving #94101) , please test.
Comment 53 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 03:10:49 UTC
Warning: you have to use XMLTV_OPTS as it was in 0.5.39 , check ebuild
with "cat $(equery w xmltv)"
committed version is 0.5.43-r3, to let anyone update.
Comment 54 Horst Schirmeier 2006-06-01 06:28:03 UTC
In-tree 0.5.43-r3 ebuild does NOT work as the tv_grab_de_tvtoday-0.5.43.diff was checked in the wrong way ($Id$ was expanded to Portage CVS revision), resulting in a patch failure.

Use cvs add -kb tv_grab_de_tvtoday-0.5.43.diff or remove the first chunk from the diff (a Portage tree grep indicates that the latter is what other Gentoo devs do).
Comment 55 Horst Schirmeier 2006-06-01 06:29:36 UTC
Created attachment 88092 [details, diff]
tv_grab_de_tvtoday-0.5.43.diff without problematic first chunk

First chunk with problematic CVS $Id$ removed.
Comment 56 Horst Schirmeier 2006-06-01 06:42:07 UTC
Disregard my comment on the Portage tree grep; I was just too stupid to use grep.
Comment 57 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 06:48:59 UTC
Changed de_tvtoday patch in cvs and changed ebuild to patch only when needed.
(xmltv_opts has noting || tv_grab_de_tvtoday)
Comment 58 Horst Schirmeier 2006-06-01 08:33:30 UTC
Works fine for me.
Comment 59 Daniel Westermann-Clark 2006-06-01 08:40:37 UTC
No problems with media-tv/xmltv-0.5.43-r3 (specifically tv_grab_na_dd, tv_sort, and tv_grep) here.

Is there any reason to run the test suite, though?  Most perl-module.eclass-based ebuilds use FEATURES="test" and a SRC_TEST="do" to skip the tests unless specifically requested.
Comment 60 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 09:46:07 UTC
In previous releases, grabbers others than na_dd (and maybe uk) weren't 
buit properly without the test, so the ebuild would have been broken 
for the 95% of the grabbers without this behaviour.

Since you are na_dd user please test if tv_grab_na_dd still needs really 
dev-perl/Class-MethodMaker previous than version 2 and report.....
MethodMaker 1.x is quite old and unmaintained.
Comment 61 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 10:12:42 UTC
Added dev-perl/Unicode-UTF8simple dep for pt users. Hopefully this bugreport will
be closed soon and in a month the new xmltv could become stable, so old and 
broken releases will be removed...
Comment 62 Leszek 2006-06-01 13:39:08 UTC
hmmm where should i put this XMLTV_OPTS var ? 
i tried to put it in make.conf and before emerge command but it always want to add a lot of unneeded dependencies:

# XMLTV_OPTS="tv_grab_uk_be" emerge xmltv -va

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild  N    ] dev-perl/perl-tk-804.027  USE="-minimal" 5,870 kB
[ebuild  N    ] dev-perl/Tk-TableMatrix-1.2.2  USE="-minimal" 310 kB
[ebuild     UD] dev-perl/Class-MethodMaker-1.11 [2.08] USE="-minimal" 41 kB
[ebuild  N    ] perl-core/CGI-3.20  USE="-minimal" 224 kB
[ebuild  N    ] virtual/perl-CGI-3.20  0 kB
[ebuild  N    ] dev-perl/HTTP-Cache-Transparent-0.7  USE="-minimal" 7 kB
[ebuild  N    ] dev-perl/Archive-Zip-1.16  USE="-minimal" 108 kB
[ebuild  N    ] dev-perl/IO-stringy-2.110  USE="-minimal" 39 kB
[ebuild  N    ] dev-perl/MIME-tools-5.420  USE="-minimal" 378 kB
[ebuild  N    ] dev-perl/MIME-Lite-3.01  USE="-minimal" 48 kB
[ebuild  N    ] dev-perl/SOAP-Lite-0.66  USE="ssl -jabber -minimal" 231 kB
[ebuild  N    ] dev-perl/HTML-LinkExtractor-0.13  USE="-minimal" 15 kB
[ebuild  N    ] app-i18n/kakasi-2.3.4  1,073 kB
[ebuild  N    ] dev-perl/Text-Kakasi-2.04  USE="-minimal" 22 kB
[ebuild  N    ] dev-perl/HTML-Element-Extended-1.16  USE="-minimal" 18 kB
[ebuild  N    ] dev-perl/HTML-TableExtract-2.08  USE="-minimal" 25 kB
[ebuild  N    ] dev-perl/Test-LongString-0.09  USE="-minimal" 6 kB
[ebuild  N    ] dev-perl/WWW-Mechanize-1.18  USE="-minimal -test" 104 kB
[ebuild     U ] media-tv/xmltv-0.5.43-r3 [0.5.43-r1] USE="-minimal" 0 kB
Comment 63 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 15:33:56 UTC
You NEED to put this var in /etc/make.conf , as last line might do.
Comment 64 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 16:36:42 UTC
SORRY, I was wrong. Some likeable joker decided that XMLTV_OPTS was 
something hackish so now ebuild can't see it (tomorrow was working).
resync, ebuild is now masked waiting for approvation of an use expanded 
variable in gentoo-dev (3 days waiting, if we're lucky, much more if 
the likeable joker has something against).
Comment 65 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 16:49:48 UTC
just to let anyone follow the xmltv destiny:
http://www.newsland.it/nr/browse/linux.gentoo.dev/22311.html
Comment 66 Leszek 2006-06-01 17:09:20 UTC
it is really strange and i cannot explain it but it seems the ebuild behaves differently when it is in my overlay.
i have portage-2.1_rc3-r4

ordi1 xmltv # cat /etc/make.conf | grep XML
XMLTV_OPTS="tv_grab_uk_be"
ordi1 xmltv # emerge xmltv -vp

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild  N    ] dev-perl/perl-tk-804.027  USE="-minimal" 5,870 kB
[ebuild  N    ] dev-perl/Tk-TableMatrix-1.2.2  USE="-minimal" 310 kB
[ebuild     UD] dev-perl/Class-MethodMaker-1.11 [2.08] USE="-minimal" 41 kB
[ebuild  N    ] perl-core/CGI-3.20  USE="-minimal" 224 kB
[ebuild  N    ] virtual/perl-CGI-3.20  0 kB
[ebuild  N    ] dev-perl/HTTP-Cache-Transparent-0.7  USE="-minimal" 7 kB
[ebuild  N    ] dev-perl/Archive-Zip-1.16  USE="-minimal" 108 kB
[ebuild  N    ] dev-perl/IO-stringy-2.110  USE="-minimal" 39 kB
[ebuild  N    ] dev-perl/MIME-tools-5.420  USE="-minimal" 378 kB
[ebuild  N    ] dev-perl/MIME-Lite-3.01  USE="-minimal" 48 kB
[ebuild  N    ] dev-perl/SOAP-Lite-0.66  USE="ssl -jabber -minimal" 231 kB
[ebuild  N    ] dev-perl/HTML-LinkExtractor-0.13  USE="-minimal" 15 kB
[ebuild  N    ] dev-perl/Unicode-UTF8simple-1.06  USE="-minimal" 88 kB
[ebuild  N    ] app-i18n/kakasi-2.3.4  1,073 kB
[ebuild  N    ] dev-perl/Text-Kakasi-2.04  USE="-minimal" 22 kB
[ebuild  N    ] dev-perl/HTML-Element-Extended-1.16  USE="-minimal" 18 kB
[ebuild  N    ] dev-perl/HTML-TableExtract-2.08  USE="-minimal" 25 kB
[ebuild  N    ] dev-perl/Test-LongString-0.09  USE="-minimal" 6 kB
[ebuild  N    ] dev-perl/WWW-Mechanize-1.18  USE="-minimal -test" 104 kB
[ebuild     U ] media-tv/xmltv-0.5.43-r3 [0.5.43-r1] USE="-minimal" 0 kB

Total size of downloads: 8,616 kB
ordi1 xmltv # cp /usr/portage/media-tv/xmltv/xmltv-0.5.43-r3.ebuild /usr/local/portage/media-tv/xmltv/
ordi1 xmltv # ebuild /usr/local/portage/media-tv/xmltv/xmltv-0.5.43-r3.ebuild digest
>>> Creating Manifest for /usr/local/portage/media-tv/xmltv
ordi1 xmltv # emerge xmltv -vp

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild     U ] media-tv/xmltv-0.5.43-r3 [0.5.43-r1] USE="-minimal" 0 kB [1]

Total size of downloads: 0 kB
Portage overlays:
 [1] /usr/local/portage
 [2] /usr/local/overlays/xgl-coffee
ordi1 xmltv #
Comment 67 Matteo Azzali (RETIRED) gentoo-dev 2006-06-01 19:42:22 UTC
Created attachment 88152 [details]
xmltv-0.5.43-r4.ebuild

xmltv-0.5.43-r4.ebuild

This is the purposed ebuild for when/if 
http://www.newsland.it/nr/browse/linux.gentoo.dev/22311.html
get a positive answer (if no answer in 3 day is positive).
Will work like XMLTV_OPTS but with TV_GRAB in /etc/make.conf
and syntax is the one of the old use flags 
(check inside ebuild , it , na_dd , na_icons uk_bleb).
also tv_check and tv_pick_cgi become use flags....
Comment 68 Leszek 2006-06-01 20:43:48 UTC
if we can have the information on tv_grabbers with use flags, why do you want to set TV_GRAB in make.conf ? Isn't it redundant ?

i tried xmltv-0.5.43-r4.ebuild, the dependencies checking is ok but it installed all the tv_grabbers.
Comment 69 Daniel Westermann-Clark 2006-06-01 22:12:35 UTC
(In reply to comment #60)
> Since you are na_dd user please test if tv_grab_na_dd still needs really 
> dev-perl/Class-MethodMaker previous than version 2 and report.....
> MethodMaker 1.x is quite old and unmaintained.

tv_grab_na_dd works fine with dev-perl/Class-MethodMaker-2.08 here.

However, I can't find this dependency anywhere that applies to this ebuild in the xmltv-0.5.43 source tree.  The only reference is in a script used to prepare a Windows executable:

[01:09:48 dwc@fortuna ~/downloads/tmp/xmltv-0.5.43]$ grep -rni MethodMaker .
./lib/exe_opt.pl:24:-add="Class::MethodMaker::scalar"
Comment 70 Matteo Azzali (RETIRED) gentoo-dev 2006-06-02 02:34:01 UTC
xmltv-0.5.43-r4.ebuild isn't to be used now. It will work only after adding 
TV_GRAB to the USE_EXPAND var in the /usr/portage/profiles/base/make.defaults
(overlay won't work for this file).  

But I can't do this modify without contacting other developers, 
so http://www.newsland.it/nr/browse/linux.gentoo.dev/22311.html
shoould have an affermative answer or no answers in 3 days.

Then TV_GRAB would substitute XMLTV_OPTS , working in a similar manner as
VIDEO_CARDS for xorg-x11.

(also the ebuild needs cleaning)
Comment 72 Matteo Azzali (RETIRED) gentoo-dev 2006-06-02 03:30:26 UTC
Created attachment 88172 [details]
xmltv-0.5.43-r4.ebuild

perfected xmltv-0.5.43-r4.ebuild , still waiting for 
http://thread.gmane.org/gmane.linux.gentoo.devel/38719/focus=38719 approval,
repoman complaints only the IUSE.invalid (ready to commit, but needs
expanded TV_GRAB var, ) and modify to profiles/use.local.desc to add tv_check
and tv_pick_cgi .
Do not test now if you don't really know what you're doing.
Comment 73 CICaesar 2006-06-02 14:37:30 UTC
(In reply to comment #68)
> if we can have the information on tv_grabbers with use flags, why do you want
> to set TV_GRAB in make.conf ? Isn't it redundant ?

I also think there shouldn't be other ways of configuring xmltv beyond USE flag, mainly because people wouldn't know about them if installing through portage without searching the web for info first
Comment 74 Matteo Azzali (RETIRED) gentoo-dev 2006-06-02 15:30:34 UTC
(in reply to comment #73)

Use flags are not an option cause it would be hackish to add 28 local flags
for one single package (I know it's easier)......

Using TV_GRAB you would read what to set doing an emerge -pv xmltv 
as you actually can do for xorg-x11 (try emerge -pv xorg-x11 ):

# emerge -pv xmltv

These are the packages that would be merged, in order:

Calculating dependencies... done!
[ebuild   R   ] media-tv/xmltv-0.5.43-r4  USE="tv_check -minimal -tv_pick_cgi" TV_GRAB="it -br -brnet -ch -de_tvtoday -dk -ee -es -fi -fr -huro -il -is -jp -na_dd -na_icons -nl -nl_wolf -no -pt -re -se_swedb -uk_be -uk_bleb -uk_rt -za" 0 kB

There's no other "sane" way to script this ebuild and in case the proposal 
will be rejected I could do nothing for xmltv insertion in poortage.
Comment 75 Matteo Azzali (RETIRED) gentoo-dev 2006-06-06 08:17:25 UTC
Comment on attachment 88172 [details]
xmltv-0.5.43-r4.ebuild

Committed in portage a new xmltv-0.5.43-r4.ebuild, using the "old" 
local use flags method, much work for nothing :-)
Comment 76 Leszek 2006-06-06 09:04:27 UTC
there is a typo in the ebuild ..

>>> Compiling source in /var/tmp/portage/xmltv-0.5.43-r4/work/xmltv-0.5.43 ...
/usr/portage/media-tv/xmltv/xmltv-0.5.43-r4.ebuild: line 165: i: command not found

do you use vim :-) ?
Comment 77 Matteo Azzali (RETIRED) gentoo-dev 2006-06-06 10:39:55 UTC
Thanks for reporting, fixed in CVS.
Comment 78 Andrew Reinartz 2006-06-06 21:02:30 UTC
Class-MethodMaker-2.08 also works just fine with na_dd for me as well ... doesn't seem to be a need for the old dependency.

(In reply to comment #69)
> (In reply to comment #60)
> > Since you are na_dd user please test if tv_grab_na_dd still needs really 
> > dev-perl/Class-MethodMaker previous than version 2 and report.....
> > MethodMaker 1.x is quite old and unmaintained.
> 
> tv_grab_na_dd works fine with dev-perl/Class-MethodMaker-2.08 here.
> 
> However, I can't find this dependency anywhere that applies to this ebuild in
> the xmltv-0.5.43 source tree.  The only reference is in a script used to
> prepare a Windows executable:
> 
> [01:09:48 dwc@fortuna ~/downloads/tmp/xmltv-0.5.43]$ grep -rni MethodMaker .
> ./lib/exe_opt.pl:24:-add="Class::MethodMaker::scalar"
> 

Comment 79 Matteo Azzali (RETIRED) gentoo-dev 2006-06-07 03:15:41 UTC
fixed in CVS. Hopefully AMD64 will keyword soon, so we can close this 70+ comments
bugreport.... ^^
Comment 80 Anders Hellgren gentoo-dev 2006-06-10 14:45:36 UTC
Created attachment 88860 [details, diff]
Move make to src_compile and use perl-module functions

Any particular reason you use make in src_install instead of in src_compile? This patch against -r4 also uses the compile and install functions from the perl-module eclass.
Comment 81 Matteo Azzali (RETIRED) gentoo-dev 2006-06-11 03:04:14 UTC
You're right, that was obsoleted actually. Also cleaned the comments for the
use flags descriptions ( no more a complete list, just the command needed
to see the complete list from use.local.desc : "equery u media-tv/xmltv" ).
In CVS.
Comment 82 Michael Cummings (RETIRED) gentoo-dev 2006-06-18 10:34:10 UTC
(In reply to comment #80)
> Created an attachment (id=88860) [edit]
> Move make to src_compile and use perl-module functions

probably should really be using perl-app, not perl-module eclass (minor differences in the result, at this point mostly in man pages)
Comment 83 Matteo Azzali (RETIRED) gentoo-dev 2006-06-29 03:56:44 UTC
I would still need to use perl-module to src_install , as perl-app has not any 
similar function. Is this you're suggesting me? (perl-app_src_compile and then
perl-module_src_install, I've tested and it works, I wish just be assicured isn't
an hack....... and you're the eclass writer)
Comment 84 Matteo Azzali (RETIRED) gentoo-dev 2006-07-01 09:10:52 UTC
Ok, I swear if I can close this bug in days, already committed in CVS
xmltv-0.5.44.ebuild . I think after a month waiting the dependancy of bug #135775
should be ignored, correct me if I'm wrong.
Comment 85 Matteo Azzali (RETIRED) gentoo-dev 2006-07-02 07:33:21 UTC
Thanks to mcummings for solving the bugs this one is depending , 
we can close this one after 80+ comments.
Comment 86 Kurt Hindenburg 2006-07-04 16:53:57 UTC
Please add ~sparc to the 44 ebuild.  I also notice there are 2 lines for na_dd dependencies...

        na_dd? ( dev-perl/HTML-TableExtract >=dev-perl/WWW-Mechanize-1.02 )
        na_dd? ( dev-perl/SOAP-Lite dev-perl/TermReadKey )

Comment 87 Matteo Azzali (RETIRED) gentoo-dev 2006-07-05 04:33:47 UTC
You're right, the line with Mechanized in it is for na_icons users, wrongly added
to all na_dd users (I still didn't found a way to mask na_icons flag to non-na_dd
users, hwever I'll correct this).

To keyword xmltv as ~sparc I'll need to ask the sparc team, I'm not allowed to
do, so I'll fill a new bugreport.