[Bug 226535] Merge Review: w3m

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Merge Review: w3m


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226535





------- Additional Comments From pnemade at redhat.com  2007-03-28 00:41 EST -------
(In reply to comment #22)
> Created an attachment (id=151033)
 --> (https://bugzilla.redhat.com/bugzilla/attachment.cgi?id=151033&action=view) [edit]
> 0.5.1-19 spec file
> 
> If you would agree with this spec file attached,
> I can approve this package.
> 
> Changes:
> * IMO "NEWS" file should be added to documents
> * License is changed from BSD to MIT.
>   Upstream http://sourceforge.net/projects/w3m/
>   declares this is MIT.
> * Now all files under doc-jp are converted to UTF-8
>   at %prep stage, the line
> --------------------------------------------
> {!?_without_utf8:for i in $RPM_BUILD_ROOT%{_mandir}/ja/man*/*;
> --------------------------------------------
>   is no longer needed, and leaving this line
>   destroys ja man file.
> * The line
> ---------------------------------------------
> rm -rf $RPM_BUILD_ROOT%{_datadir}/doc/w3m.1
> ----------------------------------------------
>    does nothing. w3m.1 is under $RPM_BUILD_DIR/w3m-0.5.1/doc* .

Thanks Very Much for this Log review.
Updated Package with your SPEC in CVS.


-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.