Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 145540 - portage sends more than one mails for one ebuild
Summary: portage sends more than one mails for one ebuild
Status: RESOLVED FIXED
Alias: None
Product: Portage Development
Classification: Unclassified
Component: Enhancement/Feature Requests (show other bugs)
Hardware: All Linux
: High enhancement (vote)
Assignee: Portage team
URL:
Whiteboard:
Keywords: InVCS
Depends on:
Blocks:
 
Reported: 2006-08-29 16:45 UTC by emerald
Modified: 2008-03-20 18:15 UTC (History)
0 users

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


Attachments
content of some mails (mails,2.79 KB, text/plain)
2006-08-30 11:09 UTC, emerald
Details

Note You need to log in before you can comment on or make changes to this bug.
Description emerald 2006-08-29 16:45:32 UTC
i use portage 2.1 logging system with email notification
to catch most of the messages during merge i set portage log level to info
for lots of ebuilds now i get 2 emails with different/separate contents
i think portage should only send one mail out for one ebuild, collecting all log/status information in one, so if there's something special to do with/for the ebuild i still get the info
something like sending one mail at the beginning of an ebuild (if there are infos) and one at the end is not really useful for most ebuilds since it mostly takes only little more for the ebuild to finish than to me to get and read the mail.
i use portage 2.1.1_rc1-r1
Comment 1 Marius Mauch (RETIRED) gentoo-dev 2006-08-30 10:42:08 UTC
Can you attach some of those mails? The mail module is supposed to send one mail per package, if it actually sends two it's a bug, but I need to see the pattern before I can investigate this.
Comment 2 emerald 2006-08-30 11:09:46 UTC
Created attachment 95498 [details]
content of some mails

actually i've seen most of those mails sent twice are from ebuilds with changed use flags/language settings,
when updating it seems to send two mails for one ebuild merged and one unmerged (last 2 mails)
...though the info from unmerging is not really interesting anymore since the ebuild is gone !?!
Comment 3 Zac Medico gentoo-dev 2006-08-30 11:34:23 UTC
(In reply to comment #2)
> when updating it seems to send two mails for one ebuild merged and one unmerged

The package being merged is separate from the package being unmerged.  The alternative is to combine two separate packages into one email.

> ...though the info from unmerging is not really interesting anymore since the
> ebuild is gone !?!

It may be that the ebuild/eclass maintainer simply needs to switch from einfo to elog for messages that aren't really interesting.
Comment 4 Marius Mauch (RETIRED) gentoo-dev 2006-08-30 19:25:25 UTC
(In reply to comment #3)
> (In reply to comment #2)
> > when updating it seems to send two mails for one ebuild merged and one unmerged
> 
> The package being merged is separate from the package being unmerged.  The
> alternative is to combine two separate packages into one email.

Hmm, tricky idea. There is no sane general solution (e.g. combining messages in log_process()) due to dependecy on $T, so this would have to be implemented in the delivery modules by (conditional?) caching and comparing the cpv. Don't really like the idea of yet another cache.
Also could cause loss of information if a cached (and therefore delayed) message is followed by a failed merge, even if that's very unlikely.

> > ...though the info from unmerging is not really interesting anymore since the
> > ebuild is gone !?!
> 
> It may be that the ebuild/eclass maintainer simply needs to switch from einfo
> to elog for messages that aren't really interesting.

Other way around: elog for important messages, einfo for routine stuff.
Comment 5 Marius Mauch (RETIRED) gentoo-dev 2008-03-03 21:00:47 UTC
This should be mostly fixed in trunk.
Comment 6 Marius Mauch (RETIRED) gentoo-dev 2008-03-20 18:14:47 UTC
This is supposed to be fixed in portage-2.2_pre5 or earlier.
Comment 7 Marius Mauch (RETIRED) gentoo-dev 2008-03-20 18:15:26 UTC
This is supposed to be fixed in portage-2.2_pre5 or earlier.