Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 192874 - skip Manifests and empty dirs logs
Summary: skip Manifests and empty dirs logs
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Infrastructure
Classification: Unclassified
Component: Git (show other bugs)
Hardware: All All
: High enhancement (vote)
Assignee: Gentoo Infrastructure
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-09-17 23:35 UTC by SpanKY
Modified: 2015-09-07 09:22 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 SpanKY gentoo-dev 2007-09-17 23:35:37 UTC
can we get the commit list tweaked so it doesnt send a notice about a directory created ?  and filter out Manifest commits just like we filter digest commits
Comment 1 Robin Johnson archtester Gentoo Infrastructure gentoo-dev Security 2007-09-18 03:41:54 UTC
Manifest and new directories should be ignored by the mailer already, not sure why they aren't being.

Pylon: pingy?
Comment 2 SpanKY gentoo-dev 2007-09-18 04:38:27 UTC
looks like Manifest diffs are ignored, but not newly added Manifests ...
Comment 3 Lars Weiler (RETIRED) gentoo-dev 2007-09-18 13:56:31 UTC
(In reply to comment #2)
> looks like Manifest diffs are ignored, but not newly added Manifests ...

Yes, new additions are handled differently than modified files.  I have to figure out how I can change that behaviour.
Comment 4 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2010-08-03 22:58:38 UTC
No comment since 2007. Not too important?
Comment 5 SpanKY gentoo-dev 2010-08-03 23:35:59 UTC
it's still a bug
Comment 6 Jeremy Olexa (darkside) (RETIRED) archtester gentoo-dev Security 2010-08-04 02:42:34 UTC
Manifest-only commits are ignored by the mailer. The only time that we see the Manifest diff on the commits ML is when repoman commits the Manifest file *with* the rest of the changed contents.

That code snippet is:
unless ( $files[0] =~ /(Manifest|digest-|userinfo.xml)/i ) {
   <mail the crap>
}

But..if we just tell it to always ignore Manifest files, that should work right?

$CHANGED_FILE  = "$TMPDIR/${FILE_PREFIX}files.changed";

It is 546 lines of perl that I will look at someday but hope that someone else beats me to it :)
Comment 7 solar (RETIRED) gentoo-dev 2010-08-04 03:23:48 UTC
Vapier,
Could you please update the summary for this bug to state if you are talking about CIA, Mailing lists or other. TIA.
Comment 8 SpanKY gentoo-dev 2010-08-04 03:33:27 UTC
when i add empty dirs to the gentoo-x86 CVS tree, e-mails are sent out to the gentoo-commits mailing list

further, i dont think sending out the diff to that list of the Manifest changes are ever useful

i imagine the same thing applies to the CIA notification

as for where the problem lies, i'll let you guys update the summary as i'm not familiar with the backend pieces
Comment 9 Christian Ruppert (idl0r) gentoo-dev 2011-02-04 23:15:54 UTC
This will be fixed as soon as we migrate to git.
Comment 10 Alex Legler (RETIRED) archtester gentoo-dev Security 2015-09-07 09:22:25 UTC
(In reply to Christian Ruppert (idl0r) from comment #9)
> This will be fixed as soon as we migrate to git.

wooho