* Re: [Mesa3d-dev] [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output
[not found] <1266791605-22944-1-git-send-email-keithp@keithp.com>
@ 2010-02-22 0:54 ` Dan Nicholson
2010-02-22 15:17 ` Brian Paul
2010-02-22 15:16 ` Brian Paul
1 sibling, 1 reply; 6+ messages in thread
From: Dan Nicholson @ 2010-02-22 0:54 UTC (permalink / raw)
To: Keith Packard; +Cc: dri-devel, mesa3d-dev
On Sun, Feb 21, 2010 at 2:33 PM, Keith Packard <keithp@keithp.com> wrote:
> The bash 'cd' command tends to emit random stuff to stdout when the
> CDPATH variable is set, so clear it to keep extra filenames from being
> emitted from the expand_archive function, which would otherwise cause
> mklib to fail.
>
> Signed-off-by: Keith Packard <keithp@keithp.com>
Congratulations on wading in to mklib, it's not a friendly place. :)
Reviewed-by: Dan Nicholson <dbn.lists@gmail.com>
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
--
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output
[not found] <1266791605-22944-1-git-send-email-keithp@keithp.com>
2010-02-22 0:54 ` [Mesa3d-dev] [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output Dan Nicholson
@ 2010-02-22 15:16 ` Brian Paul
2010-02-22 18:50 ` Keith Packard
1 sibling, 1 reply; 6+ messages in thread
From: Brian Paul @ 2010-02-22 15:16 UTC (permalink / raw)
To: Keith Packard; +Cc: dri-devel, mesa3d-dev
On Sun, Feb 21, 2010 at 3:33 PM, Keith Packard <keithp@keithp.com> wrote:
> The bash 'cd' command tends to emit random stuff to stdout when the
> CDPATH variable is set, so clear it to keep extra filenames from being
> emitted from the expand_archive function, which would otherwise cause
> mklib to fail.
>
> Signed-off-by: Keith Packard <keithp@keithp.com>
Signed-off-by: Brian Paul <brianp@vmware.com>
Please commit to both master and mesa_7_7_branch. Thanks.
-Brian
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output
2010-02-22 0:54 ` [Mesa3d-dev] [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output Dan Nicholson
@ 2010-02-22 15:17 ` Brian Paul
2010-02-22 15:42 ` Daniel Stone
2010-02-22 16:27 ` Dan Nicholson
0 siblings, 2 replies; 6+ messages in thread
From: Brian Paul @ 2010-02-22 15:17 UTC (permalink / raw)
To: Dan Nicholson; +Cc: dri-devel, mesa3d-dev
On Sun, Feb 21, 2010 at 5:54 PM, Dan Nicholson <dbn.lists@gmail.com> wrote:
> On Sun, Feb 21, 2010 at 2:33 PM, Keith Packard <keithp@keithp.com> wrote:
>> The bash 'cd' command tends to emit random stuff to stdout when the
>> CDPATH variable is set, so clear it to keep extra filenames from being
>> emitted from the expand_archive function, which would otherwise cause
>> mklib to fail.
>>
>> Signed-off-by: Keith Packard <keithp@keithp.com>
>
> Congratulations on wading in to mklib, it's not a friendly place. :)
Heh, I guess you've never had to debug libtool then. mklib is a walk
in the park by comparison.
-Brian
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output
2010-02-22 15:17 ` Brian Paul
@ 2010-02-22 15:42 ` Daniel Stone
2010-02-22 16:27 ` Dan Nicholson
1 sibling, 0 replies; 6+ messages in thread
From: Daniel Stone @ 2010-02-22 15:42 UTC (permalink / raw)
To: Brian Paul; +Cc: dri-devel, mesa3d-dev
[-- Attachment #1.1: Type: text/plain, Size: 931 bytes --]
On Mon, Feb 22, 2010 at 08:17:31AM -0700, Brian Paul wrote:
> On Sun, Feb 21, 2010 at 5:54 PM, Dan Nicholson <dbn.lists@gmail.com> wrote:
> > On Sun, Feb 21, 2010 at 2:33 PM, Keith Packard <keithp@keithp.com> wrote:
> >> The bash 'cd' command tends to emit random stuff to stdout when the
> >> CDPATH variable is set, so clear it to keep extra filenames from being
> >> emitted from the expand_archive function, which would otherwise cause
> >> mklib to fail.
> >>
> >> Signed-off-by: Keith Packard <keithp@keithp.com>
> >
> > Congratulations on wading in to mklib, it's not a friendly place. :)
>
> Heh, I guess you've never had to debug libtool then. mklib is a walk
> in the park by comparison.
In fairness though, I haven't had to debug libtool for several years
now, because it always seems to work, whereas mklib seems to encounter
new and exciting failures all the time. Oh well. :)
Cheers,
Daniel
[-- Attachment #1.2: Type: application/pgp-signature, Size: 197 bytes --]
[-- Attachment #2: Type: text/plain, Size: 345 bytes --]
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
[-- Attachment #3: Type: text/plain, Size: 161 bytes --]
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output
2010-02-22 15:17 ` Brian Paul
2010-02-22 15:42 ` Daniel Stone
@ 2010-02-22 16:27 ` Dan Nicholson
1 sibling, 0 replies; 6+ messages in thread
From: Dan Nicholson @ 2010-02-22 16:27 UTC (permalink / raw)
To: Brian Paul; +Cc: dri-devel, mesa3d-dev
On Mon, Feb 22, 2010 at 7:17 AM, Brian Paul <brian.e.paul@gmail.com> wrote:
> On Sun, Feb 21, 2010 at 5:54 PM, Dan Nicholson <dbn.lists@gmail.com> wrote:
>> On Sun, Feb 21, 2010 at 2:33 PM, Keith Packard <keithp@keithp.com> wrote:
>>> The bash 'cd' command tends to emit random stuff to stdout when the
>>> CDPATH variable is set, so clear it to keep extra filenames from being
>>> emitted from the expand_archive function, which would otherwise cause
>>> mklib to fail.
>>>
>>> Signed-off-by: Keith Packard <keithp@keithp.com>
>>
>> Congratulations on wading in to mklib, it's not a friendly place. :)
>
> Heh, I guess you've never had to debug libtool then. mklib is a walk
> in the park by comparison.
Sorry, didn't mean to imply that it's simplicity is not a virtue. On
the other hand, it doesn't get a ton of love, so it tends to be pretty
incoherent. libtool debugging is an absolute nightmare, but thankfully
it's been a long time since most people had to deal with that.
--
Dan
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output
2010-02-22 15:16 ` Brian Paul
@ 2010-02-22 18:50 ` Keith Packard
0 siblings, 0 replies; 6+ messages in thread
From: Keith Packard @ 2010-02-22 18:50 UTC (permalink / raw)
To: Brian Paul; +Cc: dri-devel, mesa3d-dev
[-- Attachment #1.1: Type: text/plain, Size: 142 bytes --]
> Please commit to both master and mesa_7_7_branch. Thanks.
mesa_7_7_branch doesn't use 'cd' in mklib.
--
keith.packard@intel.com
[-- Attachment #1.2: Type: application/pgp-signature, Size: 189 bytes --]
[-- Attachment #2: Type: text/plain, Size: 345 bytes --]
------------------------------------------------------------------------------
Download Intel® Parallel Studio Eval
Try the new software tools for yourself. Speed compiling, find bugs
proactively, and fine-tune applications for parallel performance.
See why Intel Parallel Studio got high marks during beta.
http://p.sf.net/sfu/intel-sw-dev
[-- Attachment #3: Type: text/plain, Size: 161 bytes --]
_______________________________________________
Mesa3d-dev mailing list
Mesa3d-dev@lists.sourceforge.net
https://lists.sourceforge.net/lists/listinfo/mesa3d-dev
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2010-02-22 18:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
[not found] <1266791605-22944-1-git-send-email-keithp@keithp.com>
2010-02-22 0:54 ` [Mesa3d-dev] [PATCH] bin/mklib: Clear CDPATH to avoid damaging expand_archive output Dan Nicholson
2010-02-22 15:17 ` Brian Paul
2010-02-22 15:42 ` Daniel Stone
2010-02-22 16:27 ` Dan Nicholson
2010-02-22 15:16 ` Brian Paul
2010-02-22 18:50 ` Keith Packard
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.