[Bug 473583] Review Request: WordNet - A lexical database for the english language

[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=473583





--- Comment #16 from steve <steve at lonetwin.net>  2009-05-27 07:34:02 EDT ---
(In reply to comment #15)
Thanks for your time and comments spot. Sorry about the slip-ups, this is the
first time I played around with autotools and have learned a lot in the
process.

> Well, if you're going to use a shared library here, you should do it properly,
> as a versioned .so, with proper packaging.
> 
> libWN_la_LDFLAGS = -version-number 3:0:0
> 
> will give you what you want, although, you might talk to upstream to be sure
> they're okay with the .so numbering starting there.
> 
I did mail upstream about this but got no response back. I also checked other
distributions (ubuntu and mandriva) and they seem to be using version 3.x.x for
the .so ...so, i went with that.

> Don't forget to delete the .la and .a file, your -devel wildcard is catching
> them. See why wildcards are tricky! :)

Yep ! ...and Done

> Also, you need to change a few things to ensure that the autotooling happens
> cleanly, specifically:
> 
> * BuildRequires: libtool
> * autoreconf -i instead of just autoreconf  
Done

The new spec and srpm are at:
http://www.lonetwin.net/wordnet.spec
http://lonetwin.net/wordnet-3.0-6.fc10.src.rpm

cheers,
- steve

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