From: Andre Naujoks <nautsch2@gmail.com>
To: Marc Kleine-Budde <mkl@pengutronix.de>, linux-can@vger.kernel.org
Cc: socketcan@hartkopp.net
Subject: Re: [PATCH] do not use --symlink for autoreconf
Date: Wed, 02 Jul 2014 16:09:14 +0200 [thread overview]
Message-ID: <53B4128A.80908@gmail.com> (raw)
In-Reply-To: <53B4119B.3090804@pengutronix.de>
On 02.07.2014 16:05, Marc Kleine-Budde wrote:
> On 07/02/2014 03:47 PM, Andre Naujoks wrote:
>> On 02.07.2014 15:08, Marc Kleine-Budde wrote:
>>> On 07/02/2014 03:06 PM, Marc Kleine-Budde wrote:
>>>> On 07/02/2014 02:58 PM, Marc Kleine-Budde wrote:
>>>>> On 07/02/2014 02:41 PM, Andre Naujoks wrote:
>>>>>> On 02.07.2014 14:34, Andre Naujoks wrote:
>>>>>>> On 02.07.2014 14:18, Marc Kleine-Budde wrote:
>>>>>>>> On 07/02/2014 02:12 PM, Andre Naujoks wrote:
>>>>>>>>> When packaging the can-utils, I understand, that
>>>>>>>>> the generation step for autoconf should be done
>>>>>>>>> beforehand. However using the --symlink option in
>>>>>>>>> the autoreconf step installs links instead of the
>>>>>>>>> actual files, which leads to a build failure when
>>>>>>>>> trying to build on another host with different
>>>>>>>>> autotools installed.
>>>>>>>>>
>>>>>>>>> Signed-off-by: Andre Naujoks <nautsch2@gmail.com>
>>>>>>>>
>>>>>>>> How do you package the can-utils? The usual steps
>>>>>>>> are:
>>>>>>>>
>>>>>>>> ./autogen.sh ./configure make dist
>>>>>>>
>>>>>>> with git archive. Something like
>>>>>>>
>>>>>>> git archive --prefix=can-utils-`git describe --always
>>>>>>> --dirty`/ -o can-utils-`git describe --always
>>>>>>> --dirty`.tar.gz HEAD
>>>>>>
>>>>>> ... after which I unpack, autogen.sh and repack. Which
>>>>>> is probably not the sanest way of doing this, but should
>>>>>> work.
>>>>>
>>>>> This is why you get the problems with the links....
>>>>
>>>> Try something like this instead (untested, though):
>>>>
>>>> project=can-utils-`git describe --always --dirty` git
>>>> archive --prefix=$project/ -o $project.tar HEAD tar xf
>>>> $project.tar cd $project ./autogen.sh cd .. tar -rhf
>>>> $project.tar \ $project/GNUmakefile.in $project/aclocal.m4
>>>> $project/autom4te.cache/
>>> ^^^^^^^^^^^^^^^^^^^^^^^^
>>
>> This is all nice and all, and I appreciate it, but why is the
>> --symlink parameter there in the first place? I see no benefit
>> from using it. It just makes things harder if you do it "the
>> normal way".
>
> Yes, I think the autotools/git interaction can be improved.
>
>> Another thing is, that you need a host with all build
>> dependencies resolved (i.e. configure completes) just to generate
>> an unbuild package. This might not be a problem now (or ever),
>> but why put in such a pitfall, when it is not needed?
>
> For now I've applied your patch.
Great! Thank you. :-)
>
> Thanks, Marc
>
next prev parent reply other threads:[~2014-07-02 14:15 UTC|newest]
Thread overview: 12+ messages / expand[flat|nested] mbox.gz Atom feed top
2014-07-02 12:12 [PATCH] do not use --symlink for autoreconf Andre Naujoks
2014-07-02 12:18 ` Marc Kleine-Budde
2014-07-02 12:34 ` Andre Naujoks
2014-07-02 12:41 ` Andre Naujoks
2014-07-02 12:58 ` Marc Kleine-Budde
2014-07-02 13:06 ` Marc Kleine-Budde
2014-07-02 13:07 ` Marc Kleine-Budde
2014-07-02 13:08 ` Marc Kleine-Budde
2014-07-02 13:47 ` Andre Naujoks
2014-07-02 14:05 ` Marc Kleine-Budde
2014-07-02 14:09 ` Andre Naujoks [this message]
2014-07-02 12:57 ` Marc Kleine-Budde
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=53B4128A.80908@gmail.com \
--to=nautsch2@gmail.com \
--cc=linux-can@vger.kernel.org \
--cc=mkl@pengutronix.de \
--cc=socketcan@hartkopp.net \
/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.