public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Josh Thompson <josh_thompson@ncsu.edu>
To: Ken Brown <kbrown@cornell.edu>
Cc: cygwin@cygwin.com, dev@vcl.apache.org
Subject: Re: change in handling quotes in cygwin package from 3.1.4-1 to 3.1.5-1
Date: Wed, 24 Jun 2020 13:26:28 -0400	[thread overview]
Message-ID: <2825592.AUrzYYFGSl@dvr> (raw)
In-Reply-To: <cd0e8a9f-6d61-5813-b2ae-c6fdbf1d9ed7@cornell.edu>

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

On Monday, June 22, 2020 4:11:35 PM EDT Ken Brown wrote:
> On 6/15/2020 12:21 PM, Josh Thompson wrote:
> > We recently noticed a change in double quote (") handling that is causing
> > a
> 
> > command we issue to fail.  The command is:
> [...]
> 
> > 3.1.4:
> > $ cmd.exe /c "echo \""
> > "
> > 
> > 3.1.5:
> > $ cmd.exe /c "echo \""
> > \"
> 
> I can confirm this change in behavior, and I thought it would be completely
> trivial to do a bisection to find the commit that caused it.  Unfortunately,
> it seems that the issue is somehow tied up with the fact that the toolchain
> used for building the cygwin package was upgraded shortly after the release
> of 3.1.4.
> 
> An attempt to rebuild 3.1.4 with the current toolchain fails because of some
> gcc/binutils changes.  So I applied the attached patches, which were
> applied to the Cygwin git master shortly after the release of 3.1.4, in
> order to make the build succeed.
> 
> After installing the rebuilt 3.1.4, however, the cmd.exe call above exhibits
> the "3.1.5" behavior rather than the "3.1.4" behavior.  I can't see
> anything in the patches that would explain this.  I thought maybe it was a
> compiler optimization problem, but rebuilding without optimization doesn't
> change anything.
> 
> I'm stumped.
> 
> Ken

Hi Ken,

Thanks for your testing on this and for following up.  How strange - I totally 
wouldn't have expected a toolchain upgrade to have changed how quotes get 
handled.

It sounds like it will be difficult to change the quote behavior.  So, we'll 
work to get VCL updated to handle the new behavior.

Thanks,
Josh
- -- 
- -------------------------------
Josh Thompson
VCL Developer
North Carolina State University

my GPG/PGP key can be found on pool.sks-keyservers.net

All electronic mail messages in connection with State business which
are sent to or received by this account are subject to the NC Public
Records Law and may be disclosed to third parties.
-----BEGIN PGP SIGNATURE-----

iF0EARECAB0WIQRMIdRtWXideTZDK31X8tBw1209AwUCXvOMxAAKCRBX8tBw1209
A/ZeAJ0c+FrRV+Sx5xEq5EbTGrkqrRKjggCeLmEG3K46Rx+WQ9Vhu27iPDnrFv4=
=OSZ9
-----END PGP SIGNATURE-----




  reply	other threads:[~2020-06-24 17:26 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-06-15 16:21 Josh Thompson
2020-06-22 20:11 ` Ken Brown
2020-06-24 17:26   ` Josh Thompson [this message]
2020-07-06 11:22     ` Corinna Vinschen
2020-07-06 14:14       ` Josh Thompson
2020-07-06 18:15         ` Corinna Vinschen
2020-07-21 16:45           ` Josh Thompson

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=2825592.AUrzYYFGSl@dvr \
    --to=josh_thompson@ncsu.edu \
    --cc=cygwin@cygwin.com \
    --cc=dev@vcl.apache.org \
    --cc=kbrown@cornell.edu \
    /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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).