public inbox for linux-media@vger.kernel.org
 help / color / mirror / Atom feed
From: Mauro Carvalho Chehab <mchehab@osg.samsung.com>
To: Hans Verkuil <hverkuil@xs4all.nl>
Cc: linux-media@vger.kernel.org
Subject: Re: [git:v4l-utils/master] Update INSTALL instructions and add a script to configure
Date: Sun, 14 Dec 2014 09:51:56 -0200	[thread overview]
Message-ID: <20141214095156.305ceb3e@recife.lan> (raw)
In-Reply-To: <548D774A.4030902@xs4all.nl>

Hi Hans,

Em Sun, 14 Dec 2014 12:40:58 +0100
Hans Verkuil <hverkuil@xs4all.nl> escreveu:

> On 12/14/2014 12:26 PM, Mauro Carvalho Chehab wrote:
> > This is an automatic generated email to let you know that the following patch were queued at the 
> > http://git.linuxtv.org/cgit.cgi/v4l-utils.git tree:
> > 
> > Subject: Update INSTALL instructions and add a script to configure
> > Author:  Mauro Carvalho Chehab <mchehab@osg.samsung.com>
> > Date:    Sun Dec 14 09:26:20 2014 -0200
> > 
> > Just running autoconf -vfi will override some needed configs
> > at libdvbv5-po. So, add a bootstrap.sh script and document it.
> 
> FYI: after running ./bootstrap.sh I get the diff shown below if I run 'git diff'.
> That suggests that there are autogenerated files in the git repo that do
> not belong there.

Yes, I'm aware of that. The bootstrap.sh is for now a hack to allow
building it with different versions of gettext. I suspect that there are
better ways of doing that, so I prefer to not remove the files generated
by gettextize for now.

> 
> I also see following message twice when running bootstrap:
> 
> -------
> Please create libdvbv5-po/Makevars from the template in libdvbv5-po/Makevars.template.
> You can then remove libdvbv5-po/Makevars.template.
> 
> Please run 'aclocal -I m4' to regenerate the aclocal.m4 file.
> You need aclocal from GNU automake 1.9 (or newer) to do this.
> Then run 'autoconf' to regenerate the configure file.
> 
> You might also want to copy the convenience header file gettext.h
> from the /usr/share/gettext directory into your package.
> It is a wrapper around <libintl.h> that implements the configure --disable-nls
> option.
> 
> Press Return to acknowledge the previous three paragraphs.

This is also due to the hack of running gettextize at
bootstrap.sh.

We might add a >/dev/null to avoid showing such useless messages,
although I'm trying to see if I can find a better way to implement it.

Regards,
Mauro

  reply	other threads:[~2014-12-14 11:52 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <E1Y07L2-0005cD-72@www.linuxtv.org>
2014-12-14 11:40 ` [git:v4l-utils/master] Update INSTALL instructions and add a script to configure Hans Verkuil
2014-12-14 11:51   ` Mauro Carvalho Chehab [this message]
2014-12-14 12:22     ` Mauro Carvalho Chehab

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20141214095156.305ceb3e@recife.lan \
    --to=mchehab@osg.samsung.com \
    --cc=hverkuil@xs4all.nl \
    --cc=linux-media@vger.kernel.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox