[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: [Xen-devel] [PATCH for-4.11] tools: set DEBUG_DIR from configure



On 3/28/18 2:26 AM, Roger Pau Monné wrote:
> On Tue, Mar 27, 2018 at 11:55:37AM -0500, Doug Goldstein wrote:
>> On 3/27/18 11:28 AM, Roger Pau Monné wrote:
>>> On Tue, Mar 27, 2018 at 05:20:57PM +0100, Roger Pau Monné wrote:
>>>> On Tue, Mar 27, 2018 at 06:18:08PM +0200, Olaf Hering wrote:
>>>>> On Tue, Mar 27, Roger Pau Monne wrote:
>>>>>
>>>>>> +AC_ARG_WITH([debugdir],
>>>>>> +    AS_HELP_STRING([--with-debugdir=DIR],
>>>>>> +    [Path to directory for debug symbols. [PREFIX/usr/lib/debug]]),
>>>>>> +    [debugdir_path=$withval],
>>>>>> +    [debugdir_path=$prefix/usr/lib/debug])
>>>>>
>>>>> Would that create /usr/usr/lib/debug per default?
>>>>
>>>> No, this would be: /usr/local/usr/lib/debug by default.
>>>
>>> Which maybe should be: /usr/local/lib/debug instead?
>>>
>>
>> You should use $prefix/lib/debug by default. Which would cause this to
>> map appropriately. If you have let's say /usr/bin/ls then by default it
>> would have /usr/lib/debug/usr/bin/ls.debug If you have
>> /usr/local/bin/foo then it would be /usr/local/debug/usr/local/bin/foo.debug
>                                                  ^ lib/
>>
>> Assuming that ls was built with ./configure --prefix=/usr and foo was
> 
> I guess you mean --prefix=/, using /usr would generate /usr/usr/bin/ls
> AFAICT?
> 
> Will send v2 shortly, hopefully this time the paths will be right :)
> 
> Thanks, Roger.
> 

No. The default prefix with autoconf is "/usr/local". But when Linux
distros package things up they use "/usr". See:
https://www.gnu.org/software/autoconf/manual/autoconf-2.69/html_node/Default-Prefix.html

-- 
Doug Goldstein

_______________________________________________
Xen-devel mailing list
Xen-devel@xxxxxxxxxxxxxxxxxxxx
https://lists.xenproject.org/mailman/listinfo/xen-devel

 


Rackspace

Lists.xenproject.org is hosted with RackSpace, monitoring our
servers 24x7x365 and backed by RackSpace's Fanatical Support®.