Please do not reply directly to this email. All additional comments should be made in the comments box of this bug. https://bugzilla.redhat.com/show_bug.cgi?id=485961 Simon Wesp <cassmodiah@xxxxxxxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- Flag|fedora-review? |fedora-review+ --- Comment #4 from Simon Wesp <cassmodiah@xxxxxxxxxxxxxxxxx> 2009-04-12 03:26:47 EDT --- @remi > I agree we should preserve timestamp as much as possible, but I don't > think this is a must when upstream don't care I don't think this is a must, so I agree with you; but imho it is possible with a little maintenance (in your case you solved it with sed) and the result is more beautiful. :-) And imho if you have to fix some important issues, then you can do this little thing by the way :-) * MUST: The package must meet the Packaging Guidelines O.K * MUST: A package must own all directories that it creates. If it does not create a directory that it uses, then it should require a package which does create that directory. O.K. * MUST: The License field in the package spec file must match the actual license. O.K. +-------------------+ | gmusicbrowser | | APPROVED | +-------------------+ -- Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email ------- You are receiving this mail because: ------- You are on the CC list for the bug. _______________________________________________ Fedora-package-review mailing list Fedora-package-review@xxxxxxxxxx http://www.redhat.com/mailman/listinfo/fedora-package-review