[Bug 510864] Review Request: colorize - Perl script to colorize logs

bugzilla at redhat.com bugzilla at redhat.com
Sat Jul 17 12:52:11 UTC 2010


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=510864

--- Comment #8 from manuel wolfshant <wolfy at nobugconsulting.ro> 2010-07-17 08:52:09 EDT ---
(In reply to comment #6)
> Manuel, do you think I shoulkd place
> http://web.archive.org/web/20040604132106/http://colorize.raszi.hu/downloads/colorize_0.3.4.tar.bz2
> in Source0?

To be honest, I would not do that because no one knows how long will
archive.org preserve anything AND they are not the actual upstream. I would at
most add a comment refering to the archive web site.
Now, looking again over Jason's comment, I think that preserving the original
(and now useless ) URL is not a good idea. I suggest to comment it and mention
that the original site no longer exists.


>What it imported from PLD repository already mentioned in first line of
>changelog entry. And all old entries removed by your suggestion.
In my opinion all the comments from the first entry of the changelog could be
removed. Just make it:

- Initial version, based on
ftp://ftp.icm.edu.pl/vol/rzm1/linux-pld-linux/dists/3.0/PLD/SRPMS/RPMS/colorize-0.3.4-1.src.rpm
- Add Russian summary and description

That's all that one might care for when looking at the Fedora package. If they
are really interested in the changes towards the PLD version, they can download
the PLD one and compare. As user of the package, I could not care less about
the state of the original PLD package.



> Branches: F-12 F-13 EL-5 EL-6
Thank you for asking for EPEL branches.

-- 
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.



More information about the package-review mailing list