| 1 |
On Mon, 2005-07-25 at 20:53 +0900, Jason Stubbs wrote: |
| 2 |
> On Monday 25 July 2005 16:51, Martin Schlemmer wrote: |
| 3 |
> > On Sat, 2005-07-23 at 11:18 -0400, Greg KH wrote: |
| 4 |
> > > On Sat, Jul 23, 2005 at 10:53:15AM -0400, Alec Warner wrote: |
| 5 |
> > > > Georgi Georgiev wrote: |
| 6 |
> > > > > Just to make sure I am not missing something. |
| 7 |
> > > > > |
| 8 |
> > > > > Does this cover the |
| 9 |
> > > > > |
| 10 |
> > > > > - If you are upgrading from a version of udev prior to 046 ... |
| 11 |
> > > > > - If you are upgrading from a version of udev prior to 050 ... |
| 12 |
> > > > > - If you are upgrading from a version of udev prior to 057 ... |
| 13 |
> > > > > - If you are upgrading from a version of udev prior to 059 ... |
| 14 |
> > > > > |
| 15 |
> > > > > cases automatically? I.e. *not* showing irrelevant warnings on every |
| 16 |
> > > > > upgrade/rebuild. |
| 17 |
> > > > > |
| 18 |
> > > > |
| 19 |
> > > > The writer of pkg_warn() could do this, it's still in the ebuild and |
| 20 |
> > > > could use the normal ebuild functions to determine what the user is |
| 21 |
> > > > running ( has_version() and whatnot ) and then run a case statement on that. |
| 22 |
> > > |
| 23 |
> > > Great, anyone care to send me a patch for the udev ebuild to do this so |
| 24 |
> > > not everyone sees this message? It will only get longer over time... |
| 25 |
> > > |
| 26 |
> > |
| 27 |
> > Something like this maybe? (Yes, I know using $T will be frowned upon, |
| 28 |
> > but not much else you can do. Also, might use has_version(), but that |
| 29 |
> > is more difficult to parse, and I figured you normally only want those |
| 30 |
> > for system udev ...) |
| 31 |
> |
| 32 |
> Combining the pkg_preinst and pkg_postinst parts (and removing the usage |
| 33 |
> of $T ;), that pretty much shows exactly what the proposed pkg_warn would |
| 34 |
> look like. Only difference being that it would be executed before emerging |
| 35 |
> starts. |
| 36 |
> |
| 37 |
|
| 38 |
Currently: |
| 39 |
- if everything is moved to pkg_preinst(), the message will not show at |
| 40 |
the end of the merge, so much higher chance of getting missed. |
| 41 |
- if everything is moved to pkg_postinst(), $udev_version will be the |
| 42 |
new version, and be of no use. |
| 43 |
- if you meant that this is for the pkg_warn() ... it still wont really |
| 44 |
help that much, as it will differ from before/after the update :/ |
| 45 |
|
| 46 |
|
| 47 |
-- |
| 48 |
Martin Schlemmer |
Archives