MIME-Version: 1.0
In-Reply-To: <20130301113130.GF5039@calimero.vinschen.de>
References: <CAKw7uVg_M4CYu1QDZgvvjaSqayx-ZD1qco8rOnygyzjcMAv7bg AT mail DOT gmail DOT com> <CAKw7uViChRNK1XXL1OT+8X5dFAuem3ORZ5ro4nC-Z+LV03QNHA AT mail DOT gmail DOT com> <20130301113130 DOT GF5039 AT calimero DOT vinschen DOT de>
From: Robert Pendell <shinji+cygwin AT elite-systems DOT org>
Date: Sat, 2 Mar 2013 08:31:31 -0500
Message-ID: <CAAeCd-O7h=UnsCbjvb7QKif=CZzbQBob5V7L_jbJk0D511B3Tw@mail.gmail.com>
Subject: Re: broken links in commit messages? (Re: src/winsup/cygserver ChangeLog.64bit)
To: cygwin AT cygwin DOT com
Content-Type: text/plain; charset=UTF-8
Mailing-List: contact cygwin-help AT cygwin DOT com; run by ezmlm
Precedence: bulk
Sender: cygwin-owner AT cygwin DOT com
Mail-Followup-To: cygwin AT cygwin DOT com
Content-Transfer-Encoding: 8bit

On Fri, Mar 1, 2013 at 6:31 AM, Corinna Vinschen wrote:
> On Mar  1 12:20, Václav Zeman wrote:
>> On 1 March 2013 11:39,  wrote:
>> > CVSROOT:        /cvs/src
>> > Module name:    src
>> > Branch:         cygwin-64bit-branch
>> > Changes by:     corinna AT sourceware DOT org  2013-03-01 10:39:51
>> >
>> > Modified files:
>> >         winsup/cygserver: ChangeLog.64bit
>> >
>> > Log message:
>> >         * client.cc: Revert changes to handle 32 bit processes on 64 bit
>> >         systems.
>> >         * sysv_msg.cc: Ditto.
>> >         * sysv_sem.cc: Ditto.
>> >         * sysv_shm.cc: Ditto.
>> >
>> > Patches:
>> > http://sourceware.org/cgi-bin/cvsweb.cgi/src/winsup/cygserver/ChangeLog.64bit.diff?cvsroot=src&only_with_tag=cygwin-64bit-branch&r1=1.1.2.6&r2=1.1.2.7
>> >
>> Recently, I am not able to open the diffs through links in commit
>> messages. For the one above, I get redirected to
>> http://sourceware.org/cgi-bin/cvsweb.cgi/src/winsup/cygserver/Attic/ChangeLog.64bit.diff
>> and I get error "Error:
>> src/winsup/cygserver/Attic/ChangeLog.64bit.diff: no such file or
>> directory".
>
> This seems to occur only with files in the Attic (for instance, files in
> the branch which don't exist in HEAD.  Unfortunately I don't know how
> the cvs mails are generated.  Interestingly, you can access the diff
> from the cvsweb interface:
>
> http://sourceware.org/cgi-bin/cvsweb.cgi/src/winsup/cygserver/Attic/ChangeLog.64bit.diff?r1=1.1.2.6&r2=1.1.2.7&cvsroot=src&only_with_tag=cygwin-64bit-branch
>
>
> Corinna
>
> --
> Corinna Vinschen                  Please, send mails regarding Cygwin to
> Cygwin Maintainer                 cygwin AT cygwin DOT com
> Red Hat

Looks like the urls in the diff url is missing the Attic folder in the
path location for files in Attic.  When this happens the server is
redirecting but isn't preserving the POST vars at the end of the url
hence the file not found.  Adding Attic/ to the path location behind
cygserver in the original url avoided the redirect. So it is safe to
say that cvsweb needed the POST vars but I have no idea why they are
being discarded during the redirect.

P.S. - I apologize if anything I just stated is obvious.

Robert Pendell
A perfect world is one of chaos.

--
Problem reports:       http://cygwin.com/problems.html
FAQ:                   http://cygwin.com/faq/
Documentation:         http://cygwin.com/docs.html
Unsubscribe info:      http://cygwin.com/ml/#unsubscribe-simple