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=522920 David Timms <dtimms@xxxxxxxxxxxx> changed: What |Removed |Added ---------------------------------------------------------------------------- AssignedTo|andrew@xxxxxxxxxxxxx |nobody@xxxxxxxxxxxxxxxxx --- Comment #24 from David Timms <dtimms@xxxxxxxxxxxx> 2011-07-17 09:08:44 EDT --- (In reply to comment #22) > I will review this as requested, OK, thanks, do you want to co-maintain after the package is approved ? What is your FAS username ? > - Update to latest upstream > - Change to using uniform macro's don't mix $RPM_BUILD_ROOT with %{buildroot} > - Split the desktop integration into a sub package, some users just need the > command line tnef for use with tools like mailscanner they do not need to be > saddled with a whole desktop environment Agreed, done. Not sure whether to require just the filesystem items, or the actual file managers (which I have done in this version). Updated: http://members.iinet.net.au/~timmsy/tnef/tnef.spec http://members.iinet.net.au/~timmsy/tnef/tnef-1.4.8-1.fc15.src.rpm -- 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. _______________________________________________ package-review mailing list package-review@xxxxxxxxxxxxxxxxxxxxxxx https://admin.fedoraproject.org/mailman/listinfo/package-review