Mirror of the gdb-patches mailing list
 help / color / mirror / Atom feed
From: Jonathan Larmour <jifl@eCosCentric.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: Pedro Alves <pedro@codesourcery.com>,  gdb-patches@sourceware.org
Subject: Re: Fix doc index name on Windows
Date: Fri, 26 Nov 2010 15:07:00 -0000	[thread overview]
Message-ID: <4CEFCD32.40203@eCosCentric.com> (raw)
In-Reply-To: <83d3ps6y9b.fsf@gnu.org>

Eli Zaretskii wrote:
>>From: Pedro Alves <pedro@codesourcery.com>
>>Date: Fri, 26 Nov 2010 12:03:05 +0000
>>Cc: Jonathan Larmour <jifl@ecoscentric.com>
>>
>>On Friday 26 November 2010 11:57:47, Eli Zaretskii wrote:
>>
>>>Then it's a bug in the cross-build version of makeinfo.  There's code
>>>in makeinfo/node.c:cm_node to handle the case when a file name
>>>produced from a node name clashes with a name of an existing file
>>>(produced from some other node name), due to limitations of the
>>>underlying filesystem.  What makeinfo does in that case is put all the
>>>nodes whose names map to the same file name on that single file.  I
>>>see this behavior in action in the Windows port of makeinfo 4.8, and
>>>the code which does this was written long before Texinfo 4.7 was
>>>released, so you must have it as well.
>>
>>If such code is only triggerable on some hosts only, then IMO it
>>is broken, because the resulting files will not be movable between
>>hosts (e.g., generate on Unix, unpack on Windows/NTFS/FAT/Samba, whatnot).
> 
> 
> IMO, "broken" is an exaggeration.  How many tools did you see that
> care about having their files produced on Unix be compatible with
> NTFS?  How many maintainers of GNU packages do you know who would even
> consider a possibility of inserting NTFS-related limitations into
> their codebase?

Evidently quite a few because my patch just brings GDB into line with what 
GAS, binutils, BFD, CPP, and LD's docs explicitly do. In this respect GDB 
is the odd one out.

> Usually, such problems are at best fixed for the hosts that use the
> affected filesystems.  And the cross-build environments should take
> care of these issues, because they _do_ (or should) care.

People don't use a special cross version of makeinfo. GDB doesn't try and 
invoke one, for example. Which is quite right. I don't think people would 
be keen if I submitted a patch requiring cross builders to have provided 
e.g. arm-eabi-makeinfo.

>>Is there a way to force that behaviour with a makeinfo command line
>>> switch or something of the sort?
> 
> 
> Not that I know of.  No one has ever asked for that, AFAIK.  But it
> should be trivial to add such a switch, now that I pointed to the code
> which does that.

I don't think it's good to make GDB dependent on a version of makeinfo 
which does not yet exist.

Jifl
-- 
eCosCentric Limited      http://www.eCosCentric.com/     The eCos experts
Barnwell House, Barnwell Drive, Cambridge, UK.       Tel: +44 1223 245571
Registered in England and Wales: Reg No 4422071.
------["Si fractum non sit, noli id reficere"]------       Opinions==mine


  parent reply	other threads:[~2010-11-26 15:07 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2010-11-25 17:40 Jonathan Larmour
2010-11-25 18:21 ` Eli Zaretskii
2010-11-25 18:47   ` Jonathan Larmour
2010-11-26 11:55     ` Eli Zaretskii
2010-11-26 12:03       ` Pedro Alves
2010-11-26 12:52         ` Eli Zaretskii
2010-11-26 13:05           ` Pedro Alves
2010-11-26 15:07           ` Jonathan Larmour [this message]
2010-11-26 16:15             ` Eli Zaretskii
2010-11-27  9:24               ` Andreas Schwab
2010-11-26 15:12       ` Jonathan Larmour
2010-11-26 16:08         ` Eli Zaretskii

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=4CEFCD32.40203@eCosCentric.com \
    --to=jifl@ecoscentric.com \
    --cc=eliz@gnu.org \
    --cc=gdb-patches@sourceware.org \
    --cc=pedro@codesourcery.com \
    /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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox