git@vger.kernel.org list mirror (unofficial, one of many)
 help / color / mirror / code / Atom feed
From: Philip Oakley <philipoakley@iee.email>
To: Johannes Schindelin <Johannes.Schindelin@gmx.de>
Cc: Jonathan Nieder <jrnieder@gmail.com>,
	Christian Couder <christian.couder@gmail.com>,
	Jeff King <peff@peff.net>, git <git@vger.kernel.org>,
	Christian Couder <chriscool@tuxfamily.org>
Subject: Re: Git in Outreachy?
Date: Wed, 16 Sep 2020 16:16:37 +0100	[thread overview]
Message-ID: <5be2fca8-2dbd-115e-b3bb-5783daeba4a3@iee.email> (raw)
In-Reply-To: <nycvar.QRO.7.76.6.2009060933480.56@tvgsbejvaqbjf.bet>

Hi, Sorry I've not been able to attend to the list discussions recently.

On 07/09/2020 19:49, Johannes Schindelin wrote:
> Hi Philip,
>
> On Fri, 4 Sep 2020, Philip Oakley wrote:
>
>> On 03/09/2020 07:00, Jonathan Nieder wrote:
>>> Christian Couder wrote:
>>>
>>>> I would appreciate help to find project ideas though. Are there still
>>>> scripts that are worth converting to C (excluding git-bisect.sh and
>>>> git-submodule.sh that are still worked on)? Are there worthy
>>>> refactorings or improvements that we could propose as projects?
>>> I think setting up something like snowpatch[*] to run CI on patches
>>> that have hit the mailing list but not yet hit "seen" might be a good
>>> project for an interested applicant (and I'd be interested in
>>> co-mentoring if we find a taker).
>>>
>>> Some other topics that could be interesting:
>>> - better support for handling people's name changing
>>> - making signing features such as signed push easier to use (for
>>>   example by allowing signing with SSH keys to simplify PKI) and more
>>>   useful (for example by standardizing a way to publish signed push
>>>   logs in Git)
>>> - protocol: sharing notes and branch descriptions
>>> - formats: on-disk reverse idx
>>> - obliterate
>>> - cache server to take advantage of multiple promisors+packfile URIs
>>>
>>> Jonathan
>>>
>>> [*] https://github.com/ruscur/snowpatch
>> A suggestion with high value for the Windows community
>> - mechanism to map file names between the index and the local FS, should
>> a repos file/path name already be taken, or invalid. [1]
> This suggestion keeps coming up, but I cannot help but highly doubt that
> it will prove useful in practice: if your source code contains a file
> called `aux.c`, chances are that your build system lists this file
> specifically, and it won't do at all to "magically" rename it to, say,
> `aux_.c` during checkout.

I'd disagree with that line of reasoning in the sense that if someone is
on Windows wanting to 'view' a repo that was developed on Linux, with
colons in pathnames, and filenames like aux.c we shouldn't be
deliberately de-include them just because of those file/pathname
'accidents. I accept that the build system probably won't be working for
their Windows environment (how could it be?), but, if possible, we
should be able to support them, in some positive way. In our distributed
collaborative environment we can trip over the user's  'your file / your
build' tag.

> In contrast, I think a much more useful project would be to relax the
> `core.protectNTFS` protections to cover only the files that will be
> written to disk, and not bother even checking the files excluded from a
> sparse-checkout for invalid file names on NTFS.

That's a valid base method for all the NTFS valid file and path names.

The next level could be a mechanism for path and file name adjustment,
at least for a _copy-out_ step (without ability to 'git add' / check
back in).
>
> This is trickier, of course, than meets the eye: we would still want to be
> _very_ careful to ensure that the unchecked file names will _never_ make
> it to the disk. And, slightly related, the question whether checking for
> `.git` (or `GIT~1`) would be likewise weakened, or whether that is too
> dangerous to allow even in `skip-worktree` entries.

Agree that the security aspects of `.Git` etc must still be retained.
>
> Not necessarily decisions you would want to burden a first-time
> contributor with.

True, but still worth recording as a useful Git project (and that there
are a number of nuances within it!)

--
Philip

> Ciao,
> Dscho
>
>> Philip
>>
>> [1]
>> https://github.com/git-for-windows/git/issues/2803#issuecomment-687161483
>>


  reply	other threads:[~2020-09-16 20:57 UTC|newest]

Thread overview: 44+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-08-28  6:56 Jeff King
2020-08-31  6:55 ` Christian Couder
2020-09-03  6:00   ` Jonathan Nieder
2020-09-04 14:14     ` Philip Oakley
2020-09-07 18:49       ` Johannes Schindelin
2020-09-16 15:16         ` Philip Oakley [this message]
2020-09-16 18:43           ` Johannes Schindelin
2020-09-17 14:42             ` Philip Oakley
2020-09-09 18:26     ` Taylor Blau
2020-09-10  1:39       ` Jonathan Nieder
2020-09-10  2:19         ` Taylor Blau
2020-09-16  9:12     ` Christian Couder
2020-09-16  6:42   ` Christian Couder
2020-08-31 17:41 ` Junio C Hamano
2020-08-31 18:05 ` Emily Shaffer
2020-09-01 12:51   ` Jeff King
2020-09-03  5:41     ` Jeff King
2020-09-15 17:35       ` Jeff King
2020-09-15 17:55         ` Kaartic Sivaraam
2020-09-15 18:02           ` Jeff King
2020-09-19  8:12         ` Christian Couder
2020-09-19 15:10           ` Phillip Wood
2020-09-16  8:45     ` Christian Couder
2020-09-02  4:00 ` Johannes Schindelin
2020-09-16  9:01   ` Christian Couder
2020-09-16  9:45     ` Phillip Wood
2020-09-17  9:43     ` Christian Couder
2020-09-17 10:14       ` Phillip Wood
2020-09-18  8:37         ` Christian Couder
2020-09-17 15:34       ` Elijah Newren
2020-09-18  8:42         ` Christian Couder
2020-09-27 16:59     ` Kaartic Sivaraam
2020-09-27 21:16       ` Christian Couder
2020-10-29 10:13         ` Christian Couder
2020-09-06 18:56 ` Kaartic Sivaraam
2020-09-07 18:55   ` Johannes Schindelin
2020-09-16  9:35     ` Christian Couder
2020-09-16 20:27       ` Johannes Schindelin
2020-09-19  7:40         ` Christian Couder
2020-09-20 15:06           ` Johannes Schindelin
2020-09-20 16:31   ` Kaartic Sivaraam
2020-09-21  4:22     ` Christian Couder
2020-09-21  7:59       ` Kaartic Sivaraam
2020-09-21 20:56       ` Shourya Shukla

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=5be2fca8-2dbd-115e-b3bb-5783daeba4a3@iee.email \
    --to=philipoakley@iee.email \
    --cc=Johannes.Schindelin@gmx.de \
    --cc=chriscool@tuxfamily.org \
    --cc=christian.couder@gmail.com \
    --cc=git@vger.kernel.org \
    --cc=jrnieder@gmail.com \
    --cc=peff@peff.net \
    --subject='Re: Git in Outreachy?' \
    /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

git@vger.kernel.org list mirror (unofficial, one of many)

This inbox may be cloned and mirrored by anyone:

	git clone --mirror https://public-inbox.org/git
	git clone --mirror http://ou63pmih66umazou.onion/git
	git clone --mirror http://czquwvybam4bgbro.onion/git
	git clone --mirror http://hjrcffqmbrq6wope.onion/git

	# If you have public-inbox 1.1+ installed, you may
	# initialize and index your mirror using the following commands:
	public-inbox-init -V1 git git/ https://public-inbox.org/git \
		git@vger.kernel.org
	public-inbox-index git

Example config snippet for mirrors.
Newsgroups are available over NNTP:
	nntp://news.public-inbox.org/inbox.comp.version-control.git
	nntp://7fh6tueqddpjyxjmgtdiueylzoqt6pt7hec3pukyptlmohoowvhde4yd.onion/inbox.comp.version-control.git
	nntp://ie5yzdi7fg72h7s4sdcztq5evakq23rdt33mfyfcddc5u3ndnw24ogqd.onion/inbox.comp.version-control.git
	nntp://4uok3hntl7oi7b4uf4rtfwefqeexfzil2w6kgk2jn5z2f764irre7byd.onion/inbox.comp.version-control.git
	nntp://news.gmane.io/gmane.comp.version-control.git
 note: .onion URLs require Tor: https://www.torproject.org/

code repositories for project(s) associated with this inbox:

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

AGPL code for this site: git clone https://public-inbox.org/public-inbox.git