* [PATCH] mips: use $dir for path to ldd-rewrite.sed
@ 2012-04-19 1:51 Mike Frysinger
2012-04-19 2:23 ` Carlos O'Donell
` (2 more replies)
0 siblings, 3 replies; 17+ messages in thread
From: Mike Frysinger @ 2012-04-19 1:51 UTC (permalink / raw)
To: libc-ports; +Cc: mips
The $dest variable is based on the top build dir but we need the ldd
rewrite variable to be based on the top source dir, so use $dir.
Signed-off-by: Mike Frysinger <vapier@gentoo.org>
2012-04-16 Mike Frysinger <vapier@gentoo.org>
* sysdeps/unix/sysv/linux/mips/mips64/configure.in
(ldd_rewrite_script): Change $dest to $dir.
* sysdeps/unix/sysv/linux/mips/mips64/configure: Regenerate.
---
sysdeps/unix/sysv/linux/mips/mips64/configure | 2 +-
sysdeps/unix/sysv/linux/mips/mips64/configure.in | 2 +-
2 files changed, 2 insertions(+), 2 deletions(-)
diff --git a/sysdeps/unix/sysv/linux/mips/mips64/configure b/sysdeps/unix/sysv/linux/mips/mips64/configure
index c331f98..f86f327 100644
--- a/sysdeps/unix/sysv/linux/mips/mips64/configure
+++ b/sysdeps/unix/sysv/linux/mips/mips64/configure
@@ -1,4 +1,4 @@
# This file is generated from configure.in by Autoconf. DO NOT EDIT!
# Local configure fragment for sysdeps/unix/sysv/linux/mips/mips64.
-ldd_rewrite_script=$dest/ldd-rewrite.sed
+ldd_rewrite_script=$dir/ldd-rewrite.sed
diff --git a/sysdeps/unix/sysv/linux/mips/mips64/configure.in b/sysdeps/unix/sysv/linux/mips/mips64/configure.in
index a28638a..c137ec5 100644
--- a/sysdeps/unix/sysv/linux/mips/mips64/configure.in
+++ b/sysdeps/unix/sysv/linux/mips/mips64/configure.in
@@ -2,4 +2,4 @@ sinclude(./aclocal.m4)dnl Autoconf lossage
GLIBC_PROVIDES dnl See aclocal.m4 in the top level source directory.
# Local configure fragment for sysdeps/unix/sysv/linux/mips/mips64.
-ldd_rewrite_script=$dest/ldd-rewrite.sed
+ldd_rewrite_script=$dir/ldd-rewrite.sed
--
1.7.8.5
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 1:51 [PATCH] mips: use $dir for path to ldd-rewrite.sed Mike Frysinger
@ 2012-04-19 2:23 ` Carlos O'Donell
2012-04-19 2:28 ` Mike Frysinger
2012-04-19 19:51 ` Joseph S. Myers
2012-04-22 15:38 ` Mike Frysinger
2 siblings, 1 reply; 17+ messages in thread
From: Carlos O'Donell @ 2012-04-19 2:23 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports, mips
On Wed, Apr 18, 2012 at 9:51 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> The $dest variable is based on the top build dir but we need the ldd
> rewrite variable to be based on the top source dir, so use $dir.
>
> Signed-off-by: Mike Frysinger <vapier@gentoo.org>
>
> 2012-04-16 Mike Frysinger <vapier@gentoo.org>
>
> * sysdeps/unix/sysv/linux/mips/mips64/configure.in
> (ldd_rewrite_script): Change $dest to $dir.
> * sysdeps/unix/sysv/linux/mips/mips64/configure: Regenerate.
What problem are you trying to solve? What's the use case that's
broken? I remember seeing some patches fly by about $dest vs. $dir,
but I haven't fully absorbed the change.
Cheers,
Carlos.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 2:23 ` Carlos O'Donell
@ 2012-04-19 2:28 ` Mike Frysinger
2012-04-19 3:10 ` Carlos O'Donell
0 siblings, 1 reply; 17+ messages in thread
From: Mike Frysinger @ 2012-04-19 2:28 UTC (permalink / raw)
To: Carlos O'Donell; +Cc: libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 855 bytes --]
On Wednesday 18 April 2012 22:22:45 Carlos O'Donell wrote:
> On Wed, Apr 18, 2012 at 9:51 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> > The $dest variable is based on the top build dir but we need the ldd
> > rewrite variable to be based on the top source dir, so use $dir.
> >
> > Signed-off-by: Mike Frysinger <vapier@gentoo.org>
> >
> > 2012-04-16 Mike Frysinger <vapier@gentoo.org>
> >
> > * sysdeps/unix/sysv/linux/mips/mips64/configure.in
> > (ldd_rewrite_script): Change $dest to $dir.
> > * sysdeps/unix/sysv/linux/mips/mips64/configure: Regenerate.
>
> What problem are you trying to solve? What's the use case that's
> broken? I remember seeing some patches fly by about $dest vs. $dir,
> but I haven't fully absorbed the change.
http://sourceware.org/ml/libc-alpha/2012-04/msg00588.html
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 2:28 ` Mike Frysinger
@ 2012-04-19 3:10 ` Carlos O'Donell
0 siblings, 0 replies; 17+ messages in thread
From: Carlos O'Donell @ 2012-04-19 3:10 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports
On Wed, Apr 18, 2012 at 10:28 PM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Wednesday 18 April 2012 22:22:45 Carlos O'Donell wrote:
>> On Wed, Apr 18, 2012 at 9:51 PM, Mike Frysinger <vapier@gentoo.org> wrote:
>> > The $dest variable is based on the top build dir but we need the ldd
>> > rewrite variable to be based on the top source dir, so use $dir.
>> >
>> > Signed-off-by: Mike Frysinger <vapier@gentoo.org>
>> >
>> > 2012-04-16 Mike Frysinger <vapier@gentoo.org>
>> >
>> > * sysdeps/unix/sysv/linux/mips/mips64/configure.in
>> > (ldd_rewrite_script): Change $dest to $dir.
>> > * sysdeps/unix/sysv/linux/mips/mips64/configure: Regenerate.
>>
>> What problem are you trying to solve? What's the use case that's
>> broken? I remember seeing some patches fly by about $dest vs. $dir,
>> but I haven't fully absorbed the change.
>
> http://sourceware.org/ml/libc-alpha/2012-04/msg00588.html
Thank you, that makes sense.
The patch looks good from that perspective.
I'd say check this on Friday if nobody objects.
Cheers,
Carlos.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 1:51 [PATCH] mips: use $dir for path to ldd-rewrite.sed Mike Frysinger
2012-04-19 2:23 ` Carlos O'Donell
@ 2012-04-19 19:51 ` Joseph S. Myers
2012-04-19 20:13 ` Mike Frysinger
2012-04-22 15:38 ` Mike Frysinger
2 siblings, 1 reply; 17+ messages in thread
From: Joseph S. Myers @ 2012-04-19 19:51 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports, mips
On Wed, 18 Apr 2012, Mike Frysinger wrote:
> The $dest variable is based on the top build dir but we need the ldd
> rewrite variable to be based on the top source dir, so use $dir.
Could you explain:
* what the semantics of $dest are, and where it is set;
* what the semantics of $dir are, and where it is set;
* what the semantics of $ldd_rewrite_script are (i.e. how the value is
used, which determines what sort of path is appropriate);
* how this patch has been tested (what configurations, using relative or
absolute paths with however many directory levels to whichever are
relevant of the main source directory and the ports add-on)?
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 19:51 ` Joseph S. Myers
@ 2012-04-19 20:13 ` Mike Frysinger
2012-04-19 20:21 ` Joseph S. Myers
0 siblings, 1 reply; 17+ messages in thread
From: Mike Frysinger @ 2012-04-19 20:13 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 356 bytes --]
On Thursday 19 April 2012 15:51:40 Joseph S. Myers wrote:
> On Wed, 18 Apr 2012, Mike Frysinger wrote:
> > The $dest variable is based on the top build dir but we need the ldd
> > rewrite variable to be based on the top source dir, so use $dir.
>
> Could you explain:
please see the URL i posted to carlos that has the background research
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 20:13 ` Mike Frysinger
@ 2012-04-19 20:21 ` Joseph S. Myers
2012-04-19 20:56 ` Mike Frysinger
0 siblings, 1 reply; 17+ messages in thread
From: Joseph S. Myers @ 2012-04-19 20:21 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports
On Thu, 19 Apr 2012, Mike Frysinger wrote:
> On Thursday 19 April 2012 15:51:40 Joseph S. Myers wrote:
> > On Wed, 18 Apr 2012, Mike Frysinger wrote:
> > > The $dest variable is based on the top build dir but we need the ldd
> > > rewrite variable to be based on the top source dir, so use $dir.
> >
> > Could you explain:
>
> please see the URL i posted to carlos that has the background research
That certainly doesn't say how you tested the patch (presumably in at
least one case that failed before the patch and passed afterwards, but you
don't identify what cases).
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 20:21 ` Joseph S. Myers
@ 2012-04-19 20:56 ` Mike Frysinger
2012-04-19 21:39 ` Joseph S. Myers
0 siblings, 1 reply; 17+ messages in thread
From: Mike Frysinger @ 2012-04-19 20:56 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 1004 bytes --]
On Thursday 19 April 2012 16:20:39 Joseph S. Myers wrote:
> On Thu, 19 Apr 2012, Mike Frysinger wrote:
> > On Thursday 19 April 2012 15:51:40 Joseph S. Myers wrote:
> > > On Wed, 18 Apr 2012, Mike Frysinger wrote:
> > > > The $dest variable is based on the top build dir but we need the ldd
> > > > rewrite variable to be based on the top source dir, so use $dir.
> > >
> > > Could you explain:
> > please see the URL i posted to carlos that has the background research
>
> That certainly doesn't say how you tested the patch (presumably in at
> least one case that failed before the patch and passed afterwards, but you
> don't identify what cases).
i showed how it fails today:
http://sourceware.org/ml/libc-alpha/2012-04/msg00591.html
so after i made my fix, i did the same thing and saw the correct path. i
didn't build test mips because ia64 was affected by the same issue and fixing it
there made things build fine. ergo making the same fix to mips should work.
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 20:56 ` Mike Frysinger
@ 2012-04-19 21:39 ` Joseph S. Myers
2012-04-20 3:42 ` Mike Frysinger
2012-04-22 15:45 ` Carlos O'Donell
0 siblings, 2 replies; 17+ messages in thread
From: Joseph S. Myers @ 2012-04-19 21:39 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports
On Thu, 19 Apr 2012, Mike Frysinger wrote:
> On Thursday 19 April 2012 16:20:39 Joseph S. Myers wrote:
> > On Thu, 19 Apr 2012, Mike Frysinger wrote:
> > > On Thursday 19 April 2012 15:51:40 Joseph S. Myers wrote:
> > > > On Wed, 18 Apr 2012, Mike Frysinger wrote:
> > > > > The $dest variable is based on the top build dir but we need the ldd
> > > > > rewrite variable to be based on the top source dir, so use $dir.
> > > >
> > > > Could you explain:
> > > please see the URL i posted to carlos that has the background research
> >
> > That certainly doesn't say how you tested the patch (presumably in at
> > least one case that failed before the patch and passed afterwards, but you
> > don't identify what cases).
>
> i showed how it fails today:
> http://sourceware.org/ml/libc-alpha/2012-04/msg00591.html
>
> so after i made my fix, i did the same thing and saw the correct path. i
> didn't build test mips because ia64 was affected by the same issue and fixing it
> there made things build fine. ergo making the same fix to mips should work.
I still don't see a good explanation of what all these variables are,
along the following lines:
* $dir is a directory that may be an absolute path to a directory in an
add-on, if configured using an absolute path to the add-on or a relative
path to the add-on but an absolute path to srcdir, or a relative path to
that directory from the srcdir if both configure-time paths were relative.
* $dest is a path to $dir that is always valid from the top of the build
directory (and is relative if both the source directory and the path to
any add-on were relative, absolute if either of those was absolute).
* $ldd_rewrite_script may be either absolute or relative to the toplevel
source directory, but not relative to the build directory. It is used in
the elf/ subdirectory of the source directory via $(patsubst
$(..)/%,/%,$(..)$(ldd-rewrite-script)), which means "use
$(ldd-rewrite-script) if it's an absolute path, use
../$(ldd-rewrite-script) if it's a relative path".
Is that the correct meaning of the three variables? If so, the patch is
OK, but you need to include such explanations in self-contained patch
submissions to make it clear to readers why the patch is correct without
requiring multiple messages from other threads to be followed.
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 21:39 ` Joseph S. Myers
@ 2012-04-20 3:42 ` Mike Frysinger
2012-04-20 9:27 ` Joseph S. Myers
2012-04-22 15:45 ` Carlos O'Donell
1 sibling, 1 reply; 17+ messages in thread
From: Mike Frysinger @ 2012-04-20 3:42 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 509 bytes --]
On Thursday 19 April 2012 17:38:26 Joseph S. Myers wrote:
> Is that the correct meaning of the three variables?
yes, as the referenced thread more or less said
> If so, the patch is
> OK, but you need to include such explanations in self-contained patch
> submissions to make it clear to readers why the patch is correct without
> requiring multiple messages from other threads to be followed.
the documentation on these variables is, as i pointed out in the other thread,
non-existent
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-20 3:42 ` Mike Frysinger
@ 2012-04-20 9:27 ` Joseph S. Myers
0 siblings, 0 replies; 17+ messages in thread
From: Joseph S. Myers @ 2012-04-20 9:27 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports
On Thu, 19 Apr 2012, Mike Frysinger wrote:
> > If so, the patch is
> > OK, but you need to include such explanations in self-contained patch
> > submissions to make it clear to readers why the patch is correct without
> > requiring multiple messages from other threads to be followed.
>
> the documentation on these variables is, as i pointed out in the other thread,
> non-existent
Such documentation being nonexistent in the source tree is all the more
reason why a patch submission involving these variables needs to include
an explanation directly in the submission message to make things easy for
readers of the patch submission.
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 1:51 [PATCH] mips: use $dir for path to ldd-rewrite.sed Mike Frysinger
2012-04-19 2:23 ` Carlos O'Donell
2012-04-19 19:51 ` Joseph S. Myers
@ 2012-04-22 15:38 ` Mike Frysinger
2012-04-23 9:43 ` Joseph S. Myers
2 siblings, 1 reply; 17+ messages in thread
From: Mike Frysinger @ 2012-04-22 15:38 UTC (permalink / raw)
To: libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 334 bytes --]
On Wednesday 18 April 2012 21:51:32 Mike Frysinger wrote:
> The $dest variable is based on the top build dir but we need the ldd
> rewrite variable to be based on the top source dir, so use $dir.
at this point, i don't know what the status of this patch is, so i'm just
going to let the mips maintainer merge/whatever.
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-19 21:39 ` Joseph S. Myers
2012-04-20 3:42 ` Mike Frysinger
@ 2012-04-22 15:45 ` Carlos O'Donell
2012-04-24 4:01 ` Mike Frysinger
1 sibling, 1 reply; 17+ messages in thread
From: Carlos O'Donell @ 2012-04-22 15:45 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: Mike Frysinger, libc-ports
On Thu, Apr 19, 2012 at 5:38 PM, Joseph S. Myers
<joseph@codesourcery.com> wrote:
> On Thu, 19 Apr 2012, Mike Frysinger wrote:
>
>> On Thursday 19 April 2012 16:20:39 Joseph S. Myers wrote:
>> > On Thu, 19 Apr 2012, Mike Frysinger wrote:
>> > > On Thursday 19 April 2012 15:51:40 Joseph S. Myers wrote:
>> > > > On Wed, 18 Apr 2012, Mike Frysinger wrote:
>> > > > > The $dest variable is based on the top build dir but we need the ldd
>> > > > > rewrite variable to be based on the top source dir, so use $dir.
>> > > >
>> > > > Could you explain:
>> > > please see the URL i posted to carlos that has the background research
>> >
>> > That certainly doesn't say how you tested the patch (presumably in at
>> > least one case that failed before the patch and passed afterwards, but you
>> > don't identify what cases).
>>
>> i showed how it fails today:
>> http://sourceware.org/ml/libc-alpha/2012-04/msg00591.html
>>
>> so after i made my fix, i did the same thing and saw the correct path. i
>> didn't build test mips because ia64 was affected by the same issue and fixing it
>> there made things build fine. ergo making the same fix to mips should work.
>
> I still don't see a good explanation of what all these variables are,
> along the following lines:
>
> * $dir is a directory that may be an absolute path to a directory in an
> add-on, if configured using an absolute path to the add-on or a relative
> path to the add-on but an absolute path to srcdir, or a relative path to
> that directory from the srcdir if both configure-time paths were relative.
>
> * $dest is a path to $dir that is always valid from the top of the build
> directory (and is relative if both the source directory and the path to
> any add-on were relative, absolute if either of those was absolute).
>
> * $ldd_rewrite_script may be either absolute or relative to the toplevel
> source directory, but not relative to the build directory. It is used in
> the elf/ subdirectory of the source directory via $(patsubst
> $(..)/%,/%,$(..)$(ldd-rewrite-script)), which means "use
> $(ldd-rewrite-script) if it's an absolute path, use
> ../$(ldd-rewrite-script) if it's a relative path".
>
> Is that the correct meaning of the three variables? If so, the patch is
> OK, but you need to include such explanations in self-contained patch
> submissions to make it clear to readers why the patch is correct without
> requiring multiple messages from other threads to be followed.
Mike, Joseph,
Given that these definitions are correct could someone please add an
internals page to the wiki describing them?
I assume that once this page is created that the patch should go in to
fix MIPS and IA64.
Cheers,
Carlos.
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-22 15:38 ` Mike Frysinger
@ 2012-04-23 9:43 ` Joseph S. Myers
2012-04-24 4:03 ` Mike Frysinger
0 siblings, 1 reply; 17+ messages in thread
From: Joseph S. Myers @ 2012-04-23 9:43 UTC (permalink / raw)
To: Mike Frysinger; +Cc: libc-ports
On Sun, 22 Apr 2012, Mike Frysinger wrote:
> On Wednesday 18 April 2012 21:51:32 Mike Frysinger wrote:
> > The $dest variable is based on the top build dir but we need the ldd
> > rewrite variable to be based on the top source dir, so use $dir.
>
> at this point, i don't know what the status of this patch is, so i'm just
> going to let the mips maintainer merge/whatever.
I already said it was OK if my analysis of the variables involved was
correct - and, separately, that the patch submission ought to have
contained more information (which is a point to take account of in future
patch submissions for other patches).
--
Joseph S. Myers
joseph@codesourcery.com
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-22 15:45 ` Carlos O'Donell
@ 2012-04-24 4:01 ` Mike Frysinger
2012-04-25 4:11 ` Carlos O'Donell
0 siblings, 1 reply; 17+ messages in thread
From: Mike Frysinger @ 2012-04-24 4:01 UTC (permalink / raw)
To: Carlos O'Donell; +Cc: Joseph S. Myers, libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 427 bytes --]
On Sunday 22 April 2012 11:45:42 Carlos O'Donell wrote:
> Given that these definitions are correct could someone please add an
> internals page to the wiki describing them?
where would you suggest ? i don't see any wiki page talking about ports or
addons in general.
> I assume that once this page is created that the patch should go in to
> fix MIPS and IA64.
ia64 will start off with the correct value
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-23 9:43 ` Joseph S. Myers
@ 2012-04-24 4:03 ` Mike Frysinger
0 siblings, 0 replies; 17+ messages in thread
From: Mike Frysinger @ 2012-04-24 4:03 UTC (permalink / raw)
To: Joseph S. Myers; +Cc: libc-ports
[-- Attachment #1: Type: Text/Plain, Size: 581 bytes --]
On Monday 23 April 2012 05:43:01 Joseph S. Myers wrote:
> On Sun, 22 Apr 2012, Mike Frysinger wrote:
> > On Wednesday 18 April 2012 21:51:32 Mike Frysinger wrote:
> > > The $dest variable is based on the top build dir but we need the ldd
> > > rewrite variable to be based on the top source dir, so use $dir.
> >
> > at this point, i don't know what the status of this patch is, so i'm just
> > going to let the mips maintainer merge/whatever.
>
> I already said it was OK if my analysis of the variables involved was
> correct
i've pushed it then. thanks.
-mike
[-- Attachment #2: This is a digitally signed message part. --]
[-- Type: application/pgp-signature, Size: 836 bytes --]
^ permalink raw reply [flat|nested] 17+ messages in thread
* Re: [PATCH] mips: use $dir for path to ldd-rewrite.sed
2012-04-24 4:01 ` Mike Frysinger
@ 2012-04-25 4:11 ` Carlos O'Donell
0 siblings, 0 replies; 17+ messages in thread
From: Carlos O'Donell @ 2012-04-25 4:11 UTC (permalink / raw)
To: Mike Frysinger; +Cc: Joseph S. Myers, libc-ports
On Tue, Apr 24, 2012 at 12:02 AM, Mike Frysinger <vapier@gentoo.org> wrote:
> On Sunday 22 April 2012 11:45:42 Carlos O'Donell wrote:
>> Given that these definitions are correct could someone please add an
>> internals page to the wiki describing them?
>
> where would you suggest ? i don't see any wiki page talking about ports or
> addons in general.
You're breaking new ground.
I suggest a brand new page called "Build Framework" under the
Development->Internals Documentation section.
e.g.
http://sourceware.org/glibc/wiki/#InternalsDocumentation
Add your description there about this variable there and how it's used.
>> I assume that once this page is created that the patch should go in to
>> fix MIPS and IA64.
>
> ia64 will start off with the correct value
> -mike
^ permalink raw reply [flat|nested] 17+ messages in thread
end of thread, other threads:[~2012-04-25 4:11 UTC | newest]
Thread overview: 17+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2012-04-19 1:51 [PATCH] mips: use $dir for path to ldd-rewrite.sed Mike Frysinger
2012-04-19 2:23 ` Carlos O'Donell
2012-04-19 2:28 ` Mike Frysinger
2012-04-19 3:10 ` Carlos O'Donell
2012-04-19 19:51 ` Joseph S. Myers
2012-04-19 20:13 ` Mike Frysinger
2012-04-19 20:21 ` Joseph S. Myers
2012-04-19 20:56 ` Mike Frysinger
2012-04-19 21:39 ` Joseph S. Myers
2012-04-20 3:42 ` Mike Frysinger
2012-04-20 9:27 ` Joseph S. Myers
2012-04-22 15:45 ` Carlos O'Donell
2012-04-24 4:01 ` Mike Frysinger
2012-04-25 4:11 ` Carlos O'Donell
2012-04-22 15:38 ` Mike Frysinger
2012-04-23 9:43 ` Joseph S. Myers
2012-04-24 4:03 ` Mike Frysinger
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).