public inbox for cygwin@cygwin.com
 help / color / mirror / Atom feed
From: Brian Clifton <brian@clifton.me>
To: "cygwin@cygwin.com" <cygwin@cygwin.com>
Subject: introduction, fix for npm w.r.t. git, and questions
Date: Wed, 20 Apr 2016 15:31:00 -0000	[thread overview]
Message-ID: <BLUPR03MB230DECD3A735AF3CB2225CBDF6D0@BLUPR03MB230.namprd03.prod.outlook.com> (raw)

Hi folks,

(I've never joined a mailing list so please bear with me as I learn how this works)

########
Quick intro
########
My name is Brian, I'm a developer at a hosting company in Arizona, USA and I really enjoy Cygwin. I mostly use it at home and I appreciate all of the work everyone has done. It has made my life easier for sure.

########
Git using Windows paths
########
I saw the email chain regarding Git using Windows paths; I wanted to share that I've been fighting that too. Specifically, npm will fail npm installs because it will try to use the Windows path. I submitted the following PR to fix that:
https://github.com/npm/npm/pull/12366

The PR was denied but you can see the patch made and manually apply this yourself.  Since it was rejected, I'm trying to find a better way to do this. I am considering a solution by creating a git() function in my bashrc like so:
function git {
    for var in "$@"
    do
        if [[ $var == *":\\"* ]]; then
            $var="$(cygpath $var)" #this doesn't work, but the idea is to do something like this
        fi
    done
    command git "$@"
}

I wanted to ask what you all think about this approach. Since Cygwin converts the paths, filtering any git input through cygpath makes sense, IMO. With how many folks experience issues w/ git on Cygwin, I'd like to help popularize *some* kind of fix, whatever it might be, since NPM and likely other projects are not willing to support Cygwin.

########
Quick question(s)
########
Is the website source available in git? If so, would folks be open to PRs with styling changes and/or documentation updates?

Thanks
Brian

--
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:[~2016-04-20 15:21 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2016-04-20 15:31 Brian Clifton [this message]
2016-04-20 15:44 ` Eliot Moss
2016-04-20 19:30 ` Adam Dinwoodie
2016-04-20 20:37   ` silverwind
2016-04-20 21:05     ` Adam Dinwoodie
2016-04-20 22:10       ` silverwind
2016-04-20 23:14         ` Adam Dinwoodie

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=BLUPR03MB230DECD3A735AF3CB2225CBDF6D0@BLUPR03MB230.namprd03.prod.outlook.com \
    --to=brian@clifton.me \
    --cc=cygwin@cygwin.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).