[liblouis-liblouisxml] Re: Build broken for liblouis

  • From: Simon Aittamaa <simon.aittamaa@xxxxxxxxx>
  • To: liblouis-liblouisxml@xxxxxxxxxxxxx
  • Date: Tue, 25 Nov 2014 15:07:23 +0100

Hi,

I might be missing something, but once you've found the binary, why not
check the version with a simple shell script, something along the lines of:

MAKEINFO_VERSION=`$MAKEINFO --version|head -n1|sed -ne 's/.*
\([0-9]\+\)\.[0-9]\+.*/\1/p'`
if test $MAKEINFO_VERSION -lt 5
then
    # Version less than 5, error handling goes here?
fi

Where MAKEINFO is the detected makeinfo binary. It does assume that the
--version output of makeinfo doesn't change too much between releases,
could remove 'head -n1" and  change sed to 's/^makeinfo .*
\([0-9]\+\)\.[0-9]\+.*/\1/p' too make it slightly more robust (see
'makeinfo --version' output).

Let me know if this an acceptable solution and I can submit a pull request
:)

 -- Simon

On 25 November 2014 at 14:31, Christian Egli <christian.egli@xxxxxx> wrote:

> On 11/25/2014 12:26 PM, Ken Perry wrote:
>
>> I think the better idea is to make sure the build system warns you if
>> you're missing the library and get people to install it.
>>
>
> That is a good idea and I just now looked into it. Unfortunately I could
> not find an easy way to do this. It's easy to check for a program, but not
> so easy to check for a particular version without writing some M4.
>
> Maybe I'll just make a note in the NEWS file or in the release
> announcement.
>
> Thanks
> Christian
>
>
> --
> Christian Egli
> Swiss Library for the Blind, Visually Impaired and Print Disabled
> Grubenstrasse 12, CH-8045 Zürich, Switzerland
>
> For a description of the software, to download it and links to
> project pages go to http://www.abilitiessoft.com
>

Other related posts: