Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 296722 - eix man page wrongly states order of eix-sync hooks!
Summary: eix man page wrongly states order of eix-sync hooks!
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All All
: High normal (vote)
Assignee: Martin Väth
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-12-13 13:00 UTC by Navid Zamani
Modified: 2010-01-11 19:28 UTC (History)
1 user (show)

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


Attachments
Proper eix-sync configuration file to do everything in one go. (eix-sync.conf,293 bytes, text/plain)
2009-12-13 13:05 UTC, Navid Zamani
Details
Buggy eix-sync config file. Order according to man page. (eix-sync.conf,296 bytes, text/plain)
2009-12-13 13:07 UTC, Navid Zamani
Details

Note You need to log in before you can comment on or make changes to this bug.
Description Navid Zamani 2009-12-13 13:00:47 UTC
This one took me at least a year to find. I always had problems with eix-sync and could not find out why. Now I finally read the source code of eix-sync, and found, that the documentation states, that @@ hooks come before @ hooks, while eix-sync itself has the oder flipped. (It runs @@ hooks at the very end in UpdateDiff.)

This causes eix-update not to be executed, when eix-remote is ran in a @ hook. Because it thinks that eix-remote already ran eix-update. Instead eix-remote should be ran in a @@ hook.

Reproducible: Always

Steps to Reproduce:
1. Copy the attached “eix-sync.conf” no “/etc/”.
2. Run “eix-sync”.

Actual Results:  
When eix-sync wants to run eix-update, it shows an error, stating that apparently it already wan ran in a hook.

Expected Results:  
With proper documentation, eix-remote would have been run in a @@ hook, and eix-update before it. So the problem would not have occured.
Comment 1 Navid Zamani 2009-12-13 13:05:24 UTC
Created attachment 212874 [details]
Proper eix-sync configuration file to do everything in one go.
Comment 2 Navid Zamani 2009-12-13 13:07:50 UTC
Created attachment 212875 [details]
Buggy eix-sync config file. Order according to man page.

This is the one to use, when trying to reproduce the problem.
(But a quick look at the man page, and at the source of eix-sync, will make it perfectly clear too. :)
Comment 3 Navid Zamani 2009-12-13 13:09:12 UTC
Beware, that I haven’t checked, if similar problems exist for the !!, ! and ~ hooks. When we’re at it, this should be checked too.
Comment 4 Martin Väth 2009-12-28 11:21:13 UTC
Thanks for the report. Indeed, the order in the documentation is wrong;
also the eix-remote examples were probably meant the opposite way
(IIRC, I had written them after refreshing my memory by the documentation
above; the other examples were written earlier and appear to be correct).
I will fix the documentation, not modify the behavior, i.e. the order
will remain in normal situations:
  !!/!
  ~
  emerge --sync
  @
  eix-update
  @@
  eix-diff
Comment 5 Martin Väth 2009-12-30 10:18:12 UTC
The documentation is fixed in current eix svn trunk (>=eix-0.19.2).
However, it may take quite a while for the next releaese.
Comment 6 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2010-01-11 19:28:41 UTC
(In reply to comment #5)
> The documentation is fixed in current eix svn trunk (>=eix-0.19.2).
> However, it may take quite a while for the next releaese.
> 

in cvs tree. thx