From: Joel Brobecker <brobecker@adacore.com>
To: Eli Zaretskii <eliz@gnu.org>
Cc: gdb-patches@sourceware.org
Subject: Re: [commit] Fix fnchange.lst
Date: Fri, 02 Oct 2009 15:25:00 -0000 [thread overview]
Message-ID: <20091002152521.GX10338@adacore.com> (raw)
In-Reply-To: <83ske2ow90.fsf@gnu.org>
> There are only a few of files in libdecnumber and gnulib, so I didn't
> think it was justified to remove them. They do present a real
> problem.
I know, but I am trying to qualify a release, and from my perspective,
this is a known issue that is not blocking the release. So they are
a visual distraction.
> As for .cvsignore, they shouldn't present a problem because they are
> not in the tarball. If you are running the script on the CVS sandbox,
> then I guess we need to ignore them. I will see what I can do.
Thanks. The reason why I'm running the script on a CVS sandbox, is that
I am trying to verify that things have not regressed in that department
before making the tarballs.
> > Should we also skip testsuite files?
>
> I don't think so. Any conflicts cause trouble when unpacking the
> tarball on 8+3 and some older Windows systems.
Hmmm, we may have some work to do. I remember seeing some entries
for the testsuite, except I can't remember whether they were in a
section that we care about or not.
> I agree that too much noise is bad, but do we really have a lot of
> noise? How many positives are you willing to have before they are
> noise?
>
> > What should we do with the entries in section called:
> > "The following resolve to the same SysV file names:"
>
> Nothing. We don't care about this part of doschk's output. I thought
> I made that section disappear, but it sounds like I goofed.
At the time when I ran the program, I had several pages of output
(in a 65-line terminal). It will be better once we eliminate the
SysV errors from the list. But what I'm trying to do is to automate
the task of validating the source file names for DJGPP. I suppose
I could parse the output and eliminate the known issues, but if
you can do this exclusion from the script itself (say using a switch
to control that), then it simplifies the release process. And it helps
avoiding a silly bug in my release scripts from missing something.
--
Joel
prev parent reply other threads:[~2009-10-02 15:25 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2009-09-19 9:50 Eli Zaretskii
2009-09-19 15:58 ` Joel Brobecker
2009-09-19 17:47 ` Eli Zaretskii
2009-09-20 16:11 ` Joel Brobecker
2009-09-20 17:49 ` Eli Zaretskii
2009-09-20 19:48 ` Eli Zaretskii
2009-09-21 16:56 ` Joel Brobecker
2009-10-01 17:01 ` Joel Brobecker
2009-10-02 11:43 ` Eli Zaretskii
2009-10-02 15:25 ` Joel Brobecker [this message]
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=20091002152521.GX10338@adacore.com \
--to=brobecker@adacore.com \
--cc=eliz@gnu.org \
--cc=gdb-patches@sourceware.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 a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox