This is the mail archive of the cygwin-apps@cygwin.com mailing list for the Cygwin project.


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]
Other format: [Raw text]

Re: [Review - no go] unrtf-0.19.0


* Mon 2004-02-16 Igor Pechtchanski <pechtcha <AT> cs.nyu.edu> list.cygwin-apps
* Message-Id: <Pine.GSO.4.56.0402162033160.26191 <AT> slinky.cs.nyu.edu>
| On Tue, 17 Feb 2004, Jari Aalto+mail.linux wrote:
| 
| > UnRTF is a moderately complicated converter from RTF to other formats,
| > including HTML, LaTeX, text, and PostScript. Converting to HTML, it
| > supports tables, fonts, colors, embedded images, hyperlinks, paragraph
| > alignment among other things.
| 
| One minor comment: in setup.hint, sdesc and ldesc are the same.  The above
| description looks perfect for ldesc, FWIW.

Fixed.

| Ok, here's the review:
| 
| The executable is 0 bytes...  This is a showstopper. :-)

Wow. Bug in the script, it didnät call check() and went
unnoticed from me. Fixed. 
 
| The manpage is not compressed, but at 1.8k that hardly matters (although
| compression buys about 900 bytes).

Fixed.

| Otherwise, the programs builds fine from the source and works great.  Oh,
| and when building from source, the manpage *is* compressed. :-p
|
| I'd say, once you replace the binary tarball by the one produced by
| building from source, this should be good to go.
| 	Igor

New one has been baked:

mkdir unrtf ; cd unrtf
wget -q -O - http://tierra.dyndns.org:81/cygwin/unrtf/get.sh | sh

Jari

-- 
http://tiny-tools.sourceforge.net/
Swatch @time   http://www.mir.com.my/iTime/itime.htm
               http://www.ryanthiessen.com/swatch/resources.htm
Use Licenses!  http://www.linuxjournal.com/article.php?sid=6225
Which Licence? http://www.linuxjournal.com/article.php?sid=4825
OSI Licences   http://www.opensource.org/licenses/


Index Nav: [Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav: [Date Prev] [Date Next] [Thread Prev] [Thread Next]