All of lore.kernel.org
 help / color / mirror / Atom feed
From: Bruce Dubbs <bruce.dubbs@gmail.com>
To: The development of GNU GRUB <grub-devel@gnu.org>
Subject: Re: Broken build
Date: Wed, 17 Mar 2010 19:55:53 -0500	[thread overview]
Message-ID: <4BA17A19.60301@gmail.com> (raw)
In-Reply-To: <4BA16DBB.7060704@gmail.com>

Vladimir 'φ-coder/phcoder' Serbinenko wrote:
> Carles Pina i Estany wrote:
>> Hi,
>>
>> On Mar/17/2010, Bruce Dubbs wrote:
>>   
>>> With the addition of the internationalization code, building GRUB in
>>> a separate directory fails.
>>>     
>> I'll take a look into it "soon" (next days or next week)
>>
>> Probably it's an easy thing, if someone checks it before please report
>> :-)
>>   
>         cd $(srcdir) && $(XGETTEXT) -ctranslate --from-code=utf-8 -o $@
> -f $< --keyword=_ --keyword=N_
> to
>         cd $(srcdir) && $(XGETTEXT) -ctranslate --from-code=utf-8 -o $@
> -f po/POTFILES --keyword=_ --keyword=N_
> Feel free to test & commit

I don't think so.  When generated, the makefile has:

$(srcdir)/po/$(PACKAGE).pot: po/POTFILES po/POTFILES-shell
    cd $(srcdir) && $(XGETTEXT) -ctranslate --from-code=utf-8 -o $@ -f 
$< --keyword=_ --keyword=N_

which when instantiated by make is:

../po/grub.pot: po/POTFILES po/POTFILES-shell
   cd .. && /usr/bin/xgettext -ctranslate --from-code=utf-8 \
            -o ../po/grub.pot -f po/POTFILES --keyword=_ --keyword=N_

/usr/bin/xgettext: cannot create output file "../po/grub.pot": No such 
file or directory

If I change this to:

po/grub.pot: $(srcdir)/po/POTFILES $(srcdir)/po/POTFILES-shell
   mkdir -p po
   $(XGETTEXT) -ctranslate --from-code=utf-8 -o $@ \
   -f $< --keyword=_ --keyword=N

I get the error:

   /usr/bin/xgettext: error while opening "commands/acpi.c" for reading: 
No such file or directory

We need something like:

po/grub.pot: $(srcdir)/po/POTFILES $(srcdir)/po/POTFILES-shell
   mkdir -p po
   DIR=$PWD
   cd $(srcdir) && $(XGETTEXT) -ctranslate --from-code=utf-8 \
     -o $(DIR)/$@ -f po/POTFILES --keyword=_ --keyword=N

The use of $< is wrong here because of the cd command.


   -- Bruce



  reply	other threads:[~2010-03-18  0:56 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-03-17 16:14 Broken build Bruce Dubbs
2010-03-17 23:12 ` Carles Pina i Estany
2010-03-18  0:03   ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-03-18  0:55     ` Bruce Dubbs [this message]
2010-03-26 14:33       ` Vladimir 'φ-coder/phcoder' Serbinenko
2010-03-26 14:45         ` richardvoigt
2010-03-27  4:10         ` Bruce Dubbs
     [not found] <759fb688-e363-c442-e3cd-45b7b1a029ff@stlouisintegration.com>
2018-03-26 13:30 ` broken build Wei Liu

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=4BA17A19.60301@gmail.com \
    --to=bruce.dubbs@gmail.com \
    --cc=grub-devel@gnu.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 an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.