[Bug 470508] Review Request: Ajaxterm - A web-based terminal

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


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


Jussi Lehtola <jussi.lehtola at iki.fi> changed:

           What    |Removed                     |Added
----------------------------------------------------------------------------
               Flag|fedora-review?              |fedora-review+




--- Comment #9 from Jussi Lehtola <jussi.lehtola at iki.fi>  2008-11-17 07:30:26 EDT ---
(In reply to comment #8)
> Thanks, fixed both points.
> 
> New version:
> Spec URL: http://ruben.fedorapeople.org/Ajaxterm.spec
> SRPM URL: http://ruben.fedorapeople.org/Ajaxterm-0.10-4.fc9.src.rpm

Where do you set LANG=C (or unset LANG)?  Ajaxterm only supports latin1.

Spec cleanup: Change
%dir %{_datadir}/ajaxterm
%{_datadir}/ajaxterm/*

to simply
%{_datadir}/ajaxterm


Source and license matches upstream.

****************************
The package is APPROVED.
****************************

Do the fixes mentioned above before importing to CVS.

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