[Libexif-devel] Re: assorted patches (mostly rpm build related)

[Arnaud L. <as...@la...>]

Hello,

Le Wed, Jan 01, 2003 at 09:13:43PM +0100, Daniel Resare a écrit:
> Since the latest released versions doesn't compile i decided to
> try out HEAD cvs and to my great joy it worked out nicely.

This is disturbing. What's wrong with released versions ? We need
to know...

> A few details:
> - the exif/exif directory in cvs should really be removed
> manually so that new users doesn't get confused when they
> forget to check out with -P

I don't get that. Could you be more specific ?

> - typo in exif.1 (see attachment 0)

Commited. BTW, I also added a fix, IFD was written IDF :)

> - i wrote a .spec file for exif (see attachment 1). Just add exif.spec
> to AC_OUTPUT in configure.in and to EXTRA_DIST in Makefile.am

Done.

> - patched libexif.spec to not hardcode the library version number,
> header names and include libexif.so (attachment 2)

Commited.

> - my version of automake (automake-1.6.3-3) seems to have a 'make
> distcheck' implementation that gets confused by the empty
> CONFIG_CLEAN_FILES (then it doesn't remove libexif.spec on distclean).
> Removing the parameter from Makefile.am entirely fixes the problem

Commited, too. This should be checked a little further -- Uli, Lutz ?

> - patched also libmnote specfile (attahcment 3)

Ok.

> gettextize edits configure.in and Makefile.am when it is called
> through ./autogen.sh. I see the point in having intl/ symlinked
> in from the current version, but the changing of the mentioned
> files above makes sending in patches against cvs difficult, so
> I think that those changes should be applied to cvs HEAD.

No. We support gettextize from 0.10.40 to the latest ones, and
unfortunately they don't act the same way, one modifies the files
under every circumstances, another one doesn't, and sometimes
rerunning gettextize just adds the files twice (!). So the
current solution is the best, as we don't depend on a specific
version of gettext. That way, everybody could use the cvs.

	Arnaud.