Re: [PATCH] spice-common: remove version construction

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 




----- Mensaje original -----
> 07.02.2013 21:02, Marc-André Lureau wrote:
> > ----- Mensaje original -----
> >> Version string isn't used anywhere in spice-common, and there's
> >> no version for spice-common module per se, either.  Hoever,
> >> configure.ac has this:
> >>
> >>   AC_INIT([spice-common],
> >>           [m4_esyscmd(build-aux/git-version-gen
> >>           .tarball-version)],
> >>           [spice-devel@xxxxxxxxxxxxxxxxxxxxx])
> >>
> >> But since git-version-gen script is not provided in spice-common
> >> module, multiple error messages are produced when generating
> >> configure (running autogen/autoreconf), like this:
> >>
> >>    sh: ./build-aux/git-version-gen: not found
> >>
> >> (repeated about 50 times).
> >
> > I suppose you are doing a autoreconf from a tarball directory.
> 
> Yes, after modifying ./configure.ac or some other files it is
> necessary.
> 
> > Hmm, what about spice-protocol? Do you also have the same warning?
> 
> spice-protocol has explicit version in configure.ac, like this:
> 
>   m4_define([SPICE_MAJOR], 0)
>   m4_define([SPICE_MINOR], 12)
>   m4_define([SPICE_MICRO], 2)
>   AC_INIT(spice-protocol, [SPICE_MAJOR.SPICE_MINOR.SPICE_MICRO], [],
>   spice-protocol)
> 
> > Shipping the git-version-gen is a solution too I suppose.
> 
> The thing is that this version string is not used anywhere
> in spie-common.  Spice-common has no version number per se,
> since it is not a separate/stand-alone project.

I agree. Originally, the plan was to make spice-common actually a separate library. But it needs a lot of cleanup for that to happen. So has you found out, there is no need for git-version-gen at all atm.

ack, I'll check your patch doesn't break basic stuff, and apply.
_______________________________________________
Spice-devel mailing list
Spice-devel@xxxxxxxxxxxxxxxxxxxxx
http://lists.freedesktop.org/mailman/listinfo/spice-devel



[Index of Archives]     [Linux ARM Kernel]     [Linux ARM]     [Linux Omap]     [Fedora ARM]     [IETF Annouce]     [Security]     [Bugtraq]     [Linux]     [Linux OMAP]     [Linux MIPS]     [ECOS]     [Asterisk Internet PBX]     [Linux API]     [Monitors]