Bug 314975 – eclean + sys-apps/portage-2.1.8.3: typo in deprecation warning

• Status: RESOLVED FIXED
• Product: Gentoo Linux

[Description Toralf Förster 2010-04-13 11:46:47 UTC]

change "i" into "e" :

!!! Deprication Warning: eclean.exclude.cp_all()
New portage functionality is available Please migrate code permanently



Reproducible: Always

[Comment 1 Zac Medico 2010-04-13 16:16:56 UTC]

What version of gentoolkit do you have? That's where eclean comes from.

[Comment 2 Toralf Förster 2010-04-13 16:18:48 UTC]

(In reply to comment #1)
> What version of gentoolkit do you have? That's where eclean comes from.
> 

app-portage/gentoolkit-0.3.0_rc10-r1

[Comment 3 Paul Varner (RETIRED) 2010-04-13 18:11:55 UTC]

This has already been fixed in Subversion.  Unfortunately, the spelling error wasn't noticed until after the the 0.3.0_rc10 tarball was generated:

svn log pym/gentoolkit/eclean         
------------------------------------------------------------------------
r771 | fuzzyray | 2010-04-08 16:41:55 -0500 (Thu, 08 Apr 2010) | 1 line

Fix spelling of deprecation

[Comment 4 Zac Medico 2010-04-15 16:46:56 UTC]

*** Bug 315465 has been marked as a duplicate of this bug. ***

[Comment 5 Paul Varner (RETIRED) 2010-04-21 20:02:54 UTC]

*** Bug 316445 has been marked as a duplicate of this bug. ***

[Comment 6 Toralf Förster 2010-04-27 12:02:19 UTC]

BTW the senntences "New portage functionality is available Please migrate code permanently" do miss at least one dot and/or "!" too, isn't it ?

[Comment 7 oopojo 2010-07-20 20:35:48 UTC]

Three bangs are too much. The whole warning doesn't even make much sense, really. I think it should be removed. It does not belong here. Don't scream at users in bad, typo-ridden English to have them fix another developer's code.

[Comment 8 oopojo 2010-07-20 20:36:04 UTC]

Three bangs are too much. The whole warning doesn't even make much sense, really. I think it should be removed. It does not belong here. Don't scream at users in bad, typo-ridden English to have them fix another developer's code.

[Comment 9 Boney McCracker 2010-07-21 02:00:45 UTC]

I don't know.  I've become used to it.

To me, it's now part of what defines Gentoo.

[Comment 10 Zac Medico 2010-08-20 13:10:47 UTC]

*** Bug 326417 has been marked as a duplicate of this bug. ***

[Comment 11 Zac Medico 2010-08-20 13:41:49 UTC]

*** Bug 333623 has been marked as a duplicate of this bug. ***

[Comment 12 Tanktalus 2010-08-20 17:00:30 UTC]

Really, the typo is minor.  Is this bug tracking the fix so that eclean no longer calls deprecated functionality?

[Comment 13 Boney McCracker 2010-08-20 19:25:20 UTC]

I don't want this fixed, because I have become fond of being screamed at in broken English.

[Comment 14 Bernd Feige 2010-11-05 09:44:58 UTC]

Created attachment 253261 [details]
Remove temporary workaround for missing portdb.cp_all from eclean/exclude.py

I for my part think that such displaced (and misspelled) messages don't serve gentoo well - so here's a patch that actually does the editing suggested in the file, removing the workaround altogether. I think users with versions of portage lacking cp_all should upgrade portage before calling eclean - should be a good idea anyway...
Regards,
Bernd

[Comment 15 Paul Varner (RETIRED) 2010-11-22 20:55:48 UTC]

Released in gentoolkit-0.3.0_rc11