Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 56001 - Auto Patch with Portage
Summary: Auto Patch with Portage
Status: RESOLVED INVALID
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: [OLD] Unspecified (show other bugs)
Hardware: All Linux
: High enhancement
Assignee: Portage team
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2004-07-03 16:48 UTC by Lee Trager
Modified: 2004-07-07 22:13 UTC (History)
0 users

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


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Lee Trager 2004-07-03 16:48:26 UTC
This came up in a bug report with the alsa-lib ebuild. Instead of having an error message printing out saying that the user has to manually apply a patch why not just have portage come out with a message saying something like "Portage must change /path/to/file, /path/to/another/file, and /path/to/another are you sure you want to continue?" This way the patch will be applied right away and people dont have to type in all the commands.

Reproducible: Always
Steps to Reproduce:
Comment 1 Brian Harring (RETIRED) gentoo-dev 2004-07-07 18:55:04 UTC
Err... portage instructs people to manually patch src?
Comment 2 Lee Trager 2004-07-07 19:27:28 UTC
Ya and I was thinking it would just be easier to have portage do it automaticly instead of manually.
Comment 3 Marius Mauch (RETIRED) gentoo-dev 2004-07-07 20:34:03 UTC
Where and when does it say so?
Comment 4 Lee Trager 2004-07-07 21:22:38 UTC
Look at this bug report

http://bugs.gentoo.org/show_bug.cgi?id=55750
Comment 5 Brian Harring (RETIRED) gentoo-dev 2004-07-07 22:13:39 UTC
That's a configure script spitting out info;
couple of issues-
A) no way to catch that patch as it's spit, and apply it
B) violate QA if alsa-lib goes and modifies asound.h
C) it would get utterly insane from a QA standpoint if portage stepped in and starting applying patches that the ebuild knows nothing about

I could see adding a check to alsa-lib for configure bailing, and stating "upgrade alsa-driver or your kernel sources", but what you're after violates QA and isn't even remotely easy to do :)

Closing, since (personally) I view it as a package spitting info it shouldn't be.