From: Bernhard Walle <bernhard@bwalle.de>
To: Simon Horman <horms@verge.net.au>
Cc: kexec@lists.infradead.org
Subject: Re: [PATCH 6/6] kexec-tools: Use default rpmbuild definations
Date: Thu, 25 Mar 2010 21:34:52 +0100 [thread overview]
Message-ID: <4BABC8EC.10108@bwalle.de> (raw)
In-Reply-To: <20100324113009.GH10098@verge.net.au>
Am 24.03.2010 12:30, schrieb Simon Horman:
> On Sat, Mar 20, 2010 at 10:43:49AM +0100, Bernhard Walle wrote:
>> Am 19.03.2010 21:37, schrieb Eric W. Biederman:
>>> Ameya Palande <2ameya@gmail.com> writes:
>>>
>>>> Since current mechanism for building rpm into custom directory is not working,
>>>> remove it and switch to system defaults for rpmbuild.
>>>
>>> Does RPM really bit rot that quickly?
>>>
>>> I know at the time I added the code it was the only way I could
>>> get rpm to build an rpm without being root. Something that seems
>>> desirable.
>>
>> It works here on openSUSE 11.1 when I change
>>
>> -RPMDIR=rpm
>> +RPMDIR=$(shell pwd)/rpm
>>
>> Of course only after applying the other patches...
>
> That seems reasonable to me, care to make it a proper patch?
Since 529ad47e356a8e68be9cde771af8e909a784c3fc has been applied which
removes that code, this is no longer possible. ;)
However, I'm fine with that change. The user can always create a custom
~/.rpmmacros like
%_topdir /home/bwalle/src/packages
to build the RPM in a local tree.
Regards,
Bernhard
_______________________________________________
kexec mailing list
kexec@lists.infradead.org
http://lists.infradead.org/mailman/listinfo/kexec
next prev parent reply other threads:[~2010-03-25 20:34 UTC|newest]
Thread overview: 15+ messages / expand[flat|nested] mbox.gz Atom feed top
2010-03-18 20:03 [PATCH 0/6] kexec-tools: Fix rpm build Ameya Palande
2010-03-18 20:03 ` [PATCH 1/6] kexec-tools: Fix rpm Version tag Ameya Palande
2010-03-18 20:03 ` [PATCH 2/6] kexec-tools: Use same source in rpm spec and Makefile.in Ameya Palande
2010-03-18 20:03 ` [PATCH 3/6] kexec-tools: Add missing include files in dist target Ameya Palande
2010-03-18 20:03 ` [PATCH 4/6] kexec-tools: Add kdump and kexec_test to rpm spec Ameya Palande
2010-03-18 20:03 ` [PATCH 5/6] kexec-tools: Add rpm spec file to the dist-clean target Ameya Palande
2010-03-18 20:03 ` [PATCH 6/6] kexec-tools: Use default rpmbuild definations Ameya Palande
2010-03-19 20:37 ` Eric W. Biederman
2010-03-20 9:43 ` Bernhard Walle
2010-03-22 8:56 ` Ameya Palande
2010-03-24 11:30 ` Simon Horman
2010-03-25 20:34 ` Bernhard Walle [this message]
2010-03-25 22:55 ` Simon Horman
2010-03-29 7:13 ` Anyone working on kexec for embedded powerpc? Daniel Flemström
2010-03-24 11:29 ` [PATCH 0/6] kexec-tools: Fix rpm build Simon Horman
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=4BABC8EC.10108@bwalle.de \
--to=bernhard@bwalle.de \
--cc=horms@verge.net.au \
--cc=kexec@lists.infradead.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.