git@vger.kernel.org mailing list mirror (one of many)
 help / color / mirror / code / Atom feed
From: Steffen Prohaska <prohaska@zib.de>
To: Mark Levedahl <mlevedahl@gmail.com>,
	Junio C Hamano <gitster@pobox.com>,
	Linus Torvalds <torvalds@linux-foundation.org>
Cc: Git Mailing List <git@vger.kernel.org>,
	"Shawn O. Pearce" <spearce@spearce.org>,
	Sebastian Schuberth <sschuberth@gmail.com>
Subject: Re: git on Cygwin: Not a valid object name HEAD
Date: Tue, 7 Aug 2007 21:41:24 +0200	[thread overview]
Message-ID: <07BB2580-4406-496F-8ACE-F6A03D1687BE@zib.de> (raw)
In-Reply-To: <30e4a070708071042g5623cb7ak724a8b8e588bd1da@mail.gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2716 bytes --]

The discussion below basically leads to two questions:

Is there any chance that git can be ported to cygwin's textmode?

Is there any chance that patches would be accepted that try to
do so? Even if they add "b" to fopen and O_BINARY to open, which
both are useless on Unix?

Junio, Linus?

On Aug 7, 2007, at 7:42 PM, Mark Levedahl wrote:

> On 8/7/07, Steffen Prohaska <prohaska@zib.de> wrote:
>>
>> My question is, is there any chance to handle the shell
>> activity by setting the right CYGWIN options?
>>
>
> See my other message: text mounts are best considered obsolescent if
> not deprecated, and that mode is definitely not actively developed.
> There are just too many loopholes with forks and pipes to reliably "do
> the right thing."

I read your message and I just checked the most recent installer
of cygwin (screenshot attached).

I see three choices I'm offered:
1) the path to install;
2) install for all or just me;
3) choose the default text file type.

I wouldn't call that deprecated, not even obsolenscent. To me it
looks, as if the installer offers me a real choice. Besides the
default path (which I never changed) and the decision whether I install
for all (which I would always do) I see a single real choice in this
installer -- the text file type. That's far from deprecated. I am
strongly convinced that many consider to choose DOS/text if it offers
a benefit for them, for example if they want to run CVS in cygwin.

And if the option to choose is around today, a reasonable assumption
is that cygwin installations in textmode will be around for another
couple of years.

I was convinced that git supports Windows, at least in cygwin. I am
no longer convinced. I would no longer call this Windows support --
not even in cygwin. The first four people I asked to test git on
Windows came back to me and told me that something is broken. They
didn't do anything wrong. They just used an option that cygwin
offers them during installation -- not hidden, not deprecated, no
indication that they shouldn't use this option. They or someone else
may have made the choice years ago. Their first impression was: git
doesn't work. One of them is now testing Mercurial. He explained me
today: "well, git can't even clone its own development repo. I
recognized that Mercurial is basically doing the same and it's
working fine for me."

I'd really prefer if git handled textmode (or at least refuses to
work if it detects textmode).


> Your suggested tests for non-binary mode mounts (properly #ifdef'd for
> Cygwin, possibly only enabled if specifically configured in) are a
> reasonable idea.

We could try to restrict that to git commands that have not proved
to work with textmode.

	Steffen



[-- Attachment #2.1: cygwin.png --]
[-- Type: application/applefile, Size: 70 bytes --]

[-- Attachment #2.2: cygwin.png --]
[-- Type: image/png, Size: 21751 bytes --]

[-- Attachment #3: Type: text/plain, Size: 3 bytes --]





  reply	other threads:[~2007-08-07 19:41 UTC|newest]

Thread overview: 60+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2007-08-07  9:02 git on Cygwin: Not a valid object name HEAD Sebastian Schuberth
2007-08-07 11:58 ` Johannes Schindelin
2007-08-07 12:13   ` Sebastian Schuberth
2007-08-07 13:18     ` Sebastian Schuberth
2007-08-07 14:36       ` Shawn O. Pearce
2007-08-07 14:58         ` Brian Downing
2007-08-07 15:27           ` Sebastian Schuberth
2007-08-07 17:11           ` Steffen Prohaska
2007-08-07 17:42             ` Mark Levedahl
2007-08-07 19:41               ` Steffen Prohaska [this message]
2007-08-07 21:44                 ` Linus Torvalds
2007-08-07 22:28                   ` Junio C Hamano
2007-08-08  3:47                     ` Linus Torvalds
2007-08-07 22:46                   ` Steffen Prohaska
2007-08-07 22:49                     ` [PATCH] cygwin: added fopen "b" and open O_BINARY to support cygwin's textmode Steffen Prohaska
2007-08-07 22:49                       ` [PATCH] tests: added d2u to have unix style testfiles even in textmode Steffen Prohaska
2007-08-08  3:03                         ` Linus Torvalds
2007-08-08  5:00                           ` Steffen Prohaska
2007-08-07 23:20                     ` git on Cygwin: Not a valid object name HEAD Steffen Prohaska
2007-08-08  4:25                     ` Brian Downing
2007-08-08  5:46                       ` Steffen Prohaska
2007-08-07 22:26                 ` Junio C Hamano
2007-08-08  7:55                   ` Sebastian Schuberth
2007-08-08 13:50                 ` Mark Levedahl
2007-08-08 15:51                   ` Steffen Prohaska
2007-08-08 16:41                     ` Mark Levedahl
2007-08-08 17:20                       ` Johannes Schindelin
2007-08-08 19:37                         ` Steffen Prohaska
2007-08-08 20:13                           ` Steffen Prohaska
2007-08-08 20:36                             ` Steffen Prohaska
2007-08-08 23:07                               ` Mark Levedahl
2007-08-09  4:59                                 ` Steffen Prohaska
2007-08-09  6:01                               ` Marius Storm-Olsen
2007-08-09  6:38                                 ` Steffen Prohaska
2007-08-09  6:47                                   ` Marius Storm-Olsen
2007-08-09  8:50                                 ` Johannes Schindelin
2007-08-09  9:00                                   ` Marius Storm-Olsen
2007-08-09 10:33                                     ` Steffen Prohaska
2007-08-08 21:30                           ` Johannes Schindelin
2007-08-09  6:20                             ` Steffen Prohaska
2007-08-10  6:07                               ` Torgil Svensson
2007-08-10  7:19                                 ` Steffen Prohaska
2007-08-10 10:30                                   ` Johannes Schindelin
2007-08-10 11:14                                     ` Steffen Prohaska
2007-08-10 22:31                                   ` Torgil Svensson
2007-08-11  0:43                                     ` Steffen Prohaska
2007-08-13 10:50                                       ` Johannes Schindelin
2007-08-15  0:04                                       ` Torgil Svensson
2007-08-15  5:22                                         ` Steffen Prohaska
2007-08-15  5:30                                           ` David Kastrup
2007-08-15  5:43                                             ` Steffen Prohaska
2007-08-15  6:59                                               ` David Kastrup
     [not found]                                               ` <86k5rx474o.fsf@lola.quinscape.zz>
2007-08-15  7:38                                                 ` Shawn O. Pearce
2007-08-15 12:42                                                   ` Mark Levedahl
2007-08-15 12:48                                                     ` Mark Levedahl
2007-08-15 13:06                                                       ` Steffen Prohaska
2007-08-16  0:27                                                         ` Mark Levedahl
2007-08-16  6:04                                                           ` Steffen Prohaska
2007-08-16 11:55                                                             ` Mark Levedahl
2007-08-07 15:29       ` Mark Levedahl

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

  List information: http://vger.kernel.org/majordomo-info.html

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=07BB2580-4406-496F-8ACE-F6A03D1687BE@zib.de \
    --to=prohaska@zib.de \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=mlevedahl@gmail.com \
    --cc=spearce@spearce.org \
    --cc=sschuberth@gmail.com \
    --cc=torvalds@linux-foundation.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.
Code repositories for project(s) associated with this public inbox

	https://80x24.org/mirrors/git.git

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