Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Some file names (e.g. net/config-*-linux) are incompatible with git on windows #1

Closed
arod-1 opened this Issue Dec 9, 2009 · 3 comments

Comments

Projects
None yet
3 participants

arod-1 commented Dec 9, 2009

Git on windows (msysGit) fails to create files with * in their names.
this may be a git issue. however, it's possible that it would be good to solve this problem in webmin (i.e. put them in a different name in the repo, and have the install procedure rename them, or maybe use different name altogether)

arod-1 commented Dec 9, 2009

tested with cygwin git on windows: same issue.

Wouldn't be best place to start be to talk to the developers of msysGit and see if they can solve this issue, since it seems to be unique to their version of Git? Insofar as Git is to be considered cross-platform, it should be expected to handle files from foreign operating systems. So it seems that this may be a use case they simply haven't run into before, and may be happy to fix.

@jcameron jcameron pushed a commit that referenced this issue Mar 7, 2014

@alktlr alktlr Merge pull request #1 from webmin/master
Get latest stuff
4f2465c

@jcameron jcameron pushed a commit that referenced this issue Mar 10, 2016

@nielosz nielosz Merge pull request #1 from nielosz/Handle-Pipe-commands
Handle pipe commands
5db95b1

@jcameron jcameron pushed a commit that referenced this issue Apr 6, 2016

@bram96 bram96 Merge pull request #1 from bram96/bram96-fix-letsencrypt
Fix letsencrypt CA
ad9c2ec

@jcameron jcameron pushed a commit that referenced this issue May 7, 2017

@ymulleneers ymulleneers Merge pull request #1 from ymulleneers/ymulleneers-patch-1
Update sorttable.js
f046efe

@jcameron jcameron pushed a commit that referenced this issue May 9, 2017

@gnadelwartz gnadelwartz Merge pull request #1 from gnadelwartz/firewall-f2b-warn
merge branch wadd arning about fail2ban firewalll rules detected
0b416e0

@jcameron jcameron pushed a commit that referenced this issue May 13, 2017

@gnadelwartz gnadelwartz Merge pull request #1 from gnadelwartz/patch-2
fix my typos
ca6739c

@jcameron jcameron pushed a commit that referenced this issue May 24, 2017

@gnadelwartz gnadelwartz Merge pull request #1 from gnadelwartz/majordomo2
display list values in overview
e7c4c3a
Collaborator

swelljoe commented Jun 12, 2017 edited

This has been fixed by changing all * filenames to use ALL, instead, a couple of versions back. I've confirmed it can be cloned on Windows, but haven't done much else with the source, as I don't usually work on Windows. Feel free to open a new issue if there are any other problems with checkouts on Windows.

I know this is a really old issue, but I see that Perl has a great Windows experience these days, so there's no reason Webmin shouldn't be nice to develop on Windows too (well, except for all the other weird things about Windows that make it hard...but, we'll cross those bridges when we come to them).

@swelljoe swelljoe closed this Jun 12, 2017

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment