public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Adam Dinwoodie <Adam.Dinwoodie@metaswitch.com>
To: Jochen Wiedmann <jochen.wiedmann@softwareag.com>,
	"cygwin@cygwin.com"	<cygwin@cygwin.com>
Subject: RE: CygWin SVN should identify as CygWin
Date: Tue, 07 Aug 2012 15:08:00 -0000	[thread overview]
Message-ID: <CE9C056E12502146A72FD81290379E9A43657DAA@ENFIRHMBX1.datcon.co.uk> (raw)
In-Reply-To: <5021233D.1010305@softwareag.com>

Jochen Wiedmann wrote:
>On 07.08.2012 13:15, Adam Dinwoodie wrote:
>> This isn't Subversion's responsibility; the problem is more general: how do you
>> tell if the version of awk, sed or vim are Cygwin ones or not (or ones compiled
>> containing a specific patch, or built on a particular day, or any other of a
>> myriad of different things that could make a difference to an executable's
>> behaviour)?
>
>I don't know about "more general". However, I know very well that-
>there's a particular project (Maven Release Plugin), which has this very-
>problem with svn, not with awk, sed, or whatever. And I'd like to
>fix that specific problem, not eliminate hunger in the world, or do-
>whatever more general. To achieve that, I've pointed out a non-intrusive-
>and harmless change in CygWin SVN, which might help to resolve that problem.

Your definition of "harmless" does not necessarily match that of the package
maintainer's. You're proposing a Cygwin-specific change from the original
source code. Every such change means extra work for package maintainers, both
to make the change in the first place, then to keep making the change every
time the source code is updated.

>And, besides, your proposed solution won't work: I could, of course,
>use "which", or "where" to deduce the location of "svn", but what would
>that tell me. Assuming, I get "/usr/bin/svn", then I'd know that "which"
>is a CygWin binary (because it emits a CygWin path), but what's got that
>to do with svn? The fact that it resides in the CygWin bin directory
>doesn't mean it is also a CygWin binary.

So you want a solution that will allow you to tell when someone's been hacking
around in Cygwin's bin directory? If you're going to overwrite things in the
bin directory, I think that's something you have to do at your own risk and
knowing what you're doing. Putting executables that don't play nicely with
Cygwin in Cygwin's bin directory seems like an exceptionally bad idea however
you count it.

I think you're looking for the wrong solution. If someone is having trouble
with Maven because they've put Cygwin's bin directory in their path, then they
should clean up their path: Cygwin doesn't put itself in the path exactly
because it can cause problems like this.

--
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

  reply	other threads:[~2012-08-07 15:04 UTC|newest]

Thread overview: 12+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <5020CEC7.3010703@softwareag.com>
2012-08-07  9:47 ` marco atzeri
2012-08-07 10:07   ` Jochen Wiedmann
2012-08-07 10:37     ` Andrey Repin
2012-08-07 10:41       ` Jochen Wiedmann
2012-08-07 11:25         ` Adam Dinwoodie
2012-08-07 14:20           ` Jochen Wiedmann
2012-08-07 15:08             ` Adam Dinwoodie [this message]
2012-08-07 15:25             ` Larry Hall (Cygwin)
2012-08-07 17:15             ` Andrey Repin
2012-08-08 16:30               ` Jochen Wiedmann
2012-08-07 14:35         ` Andrey Repin
2012-08-08 16:58 ` David Rothenberger

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=CE9C056E12502146A72FD81290379E9A43657DAA@ENFIRHMBX1.datcon.co.uk \
    --to=adam.dinwoodie@metaswitch.com \
    --cc=cygwin@cygwin.com \
    --cc=jochen.wiedmann@softwareag.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;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).