Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 398809 - Migrate ebuilds to new cdrom.eclass
Summary: Migrate ebuilds to new cdrom.eclass
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: New packages (show other bugs)
Hardware: All Linux
: Normal normal (vote)
Assignee: Ulrich Müller
URL: http://archives.gentoo.org/gentoo-dev...
Whiteboard:
Keywords:
Depends on: 398799 398801
Blocks:
  Show dependency tree
 
Reported: 2012-01-13 20:49 UTC by Ulrich Müller
Modified: 2012-03-01 07:04 UTC (History)
2 users (show)

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


Attachments
Discussion in #gentoo-dev (irc.log,1.79 KB, text/plain)
2012-01-13 20:49 UTC, Ulrich Müller
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Ulrich Müller gentoo-dev 2012-01-13 20:49:13 UTC
Created attachment 298875 [details]
Discussion in #gentoo-dev

The cdrom_* functions have been split out from eutils.eclass into a new cdrom.eclass. All ebuilds using these functions should be changed as follows:
- Add cdrom to inherit line
- Optionally, PROPERTIES=interactive can be removed because the eclass
  already sets it

I'd commit these changed myself, but there were some objections against it, see attached IRC log.

When all ebuilds are done, I'll remove the functions from eutils.eclass.


List of all affected ebuilds in Portage tree:

cjk:
  app-i18n/atokx2
  app-i18n/atokx3

calchan, games:
  games-rpg/nwn-data
  games-rpg/sacred-gold

tristan, games:
  games-action/heretic2
  games-fps/etqw-data
  games-fps/postal2

vapier, games:
  games-fps/unreal-tournament-goty
  games-fps/unreal-tournament
  games-fps/unreal

games:
  games-action/d1x-rebirth
  games-action/descent2-data
  games-action/descent3
  games-action/fakk2
  games-action/heavygear2
  games-action/postalplus
  games-action/rune
  games-fps/doom3-data
  games-fps/doom3-roe
  games-fps/duke3d-data
  games-fps/quake1-data
  games-fps/quake2-data
  games-fps/quake3-data
  games-fps/quake3-teamarena
  games-fps/quake4-data
  games-fps/soldieroffortune
  games-fps/tribes2
  games-fps/ut2003-data
  games-fps/ut2004-data
  games-strategy/coldwar
  games-strategy/darwinia
  games-strategy/dominions2
  games-strategy/heroes3
  games-strategy/smac
  games-strategy/wargus
  games-strategy/x2
Comment 1 Tristan Heaven (RETIRED) gentoo-dev 2012-01-13 21:09:36 UTC
Setting PROPERTIES="interactive" in the eclass is not such a good idea. Some ebuilds are only interactive with USE=cdinstall.
Comment 2 Ulrich Müller gentoo-dev 2012-01-13 22:37:11 UTC
(In reply to comment #1)
> Setting PROPERTIES="interactive" in the eclass is not such a good idea.
> Some ebuilds are only interactive with USE=cdinstall.

Good point, I wasn't aware of this. I've removed the line from the eclass.
Comment 3 Ulrich Müller gentoo-dev 2012-01-15 14:22:43 UTC
(In reply to comment #2)
> > Setting PROPERTIES="interactive" in the eclass is not such a good idea.
> > Some ebuilds are only interactive with USE=cdinstall.
> 
> Good point, I wasn't aware of this. I've removed the line from the eclass.

Added again, but only conditionally executed, as suggested in -dev ML:

<http://sources.gentoo.org/cgi-bin/viewvc.cgi/gentoo-x86/eclass/cdrom.eclass?r1=1.2&r2=1.3>
Comment 4 Ulrich Müller gentoo-dev 2012-01-16 19:51:48 UTC
The following are done, with permission from vapier and mr_bones_:
  games-action/d1x-rebirth
  games-action/descent2-data
  games-action/descent3
  games-action/fakk2
  games-action/heavygear2
  games-action/postalplus
  games-action/rune
  games-fps/doom3-data
  games-fps/doom3-roe
  games-fps/duke3d-data
  games-fps/quake1-data
  games-fps/quake2-data
  games-fps/quake3-data
  games-fps/quake3-teamarena
  games-fps/quake4-data
  games-fps/soldieroffortune
  games-fps/tribes2
  games-fps/unreal
  games-fps/unreal-tournament
  games-fps/unreal-tournament-goty
  games-fps/ut2003-data
  games-fps/ut2004-data
  games-strategy/coldwar
  games-strategy/darwinia
  games-strategy/dominions2
  games-strategy/heroes3
  games-strategy/smac
  games-strategy/wargus
  games-strategy/x2

Still missing:

cjk:
  app-i18n/atokx2
  app-i18n/atokx3

calchan, games:
  games-rpg/nwn-data
  games-rpg/sacred-gold

tristan, games:
  games-action/heretic2
  games-fps/etqw-data
  games-fps/postal2
Comment 5 Ulrich Müller gentoo-dev 2012-01-16 20:19:06 UTC
>   games-action/heretic2
>   games-fps/etqw-data
>   games-fps/postal2

Done, with permission from tristan.
Comment 6 Ulrich Müller gentoo-dev 2012-01-19 11:13:38 UTC
@CJK team, any objections if I commit the changes (as outlined in comment #0) for app-i18n/atokx{2,3} myself?
Comment 7 SpanKY gentoo-dev 2012-01-24 02:51:58 UTC
cjk guys tend to have high latency.  just do it.
Comment 8 Ulrich Müller gentoo-dev 2012-01-26 21:48:15 UTC
(In reply to comment #6)
> @CJK team, any objections if I commit the changes (as outlined in comment #0)
> for app-i18n/atokx{2,3} myself?

Seems not. Done.
Comment 9 Ulrich Müller gentoo-dev 2012-01-31 06:50:37 UTC
I've removed the cdrom functions from eutils.eclass. For the time being, eutils inherits cdrom; this can be removed as soon as the last two packages are migrated:

calchan, games:
  games-rpg/nwn-data
  games-rpg/sacred-gold
Comment 10 Ulrich Müller gentoo-dev 2012-01-31 07:04:03 UTC
(In reply to comment #9)
> I've removed the cdrom functions from eutils.eclass. For the time being, eutils
> inherits cdrom;

Reverted. This can't be done because of PROPERTIES="interactive" in cdrom.eclass.
Comment 11 Ulrich Müller gentoo-dev 2012-02-15 23:27:20 UTC
(In reply to comment #9)
> calchan, games:
>   games-rpg/nwn-data
>   games-rpg/sacred-gold

@Calchan: Any estimate when these will be fixed?
Comment 12 Ulrich Müller gentoo-dev 2012-03-01 07:04:41 UTC
(In reply to comment #11)
> (In reply to comment #9)
> > calchan, games:
> >   games-rpg/nwn-data
> >   games-rpg/sacred-gold
> 
> @Calchan: Any estimate when these will be fixed?

No answer, so I've fixed these myself.

Finally, cdrom functions removed from eutils.eclass.