New issue

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

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

generate-load-scripts case sensitivity #2547

Closed
TheAngryByrd opened this Issue Jul 20, 2017 · 7 comments

Comments

Projects
None yet
4 participants
@TheAngryByrd
Contributor

TheAngryByrd commented Jul 20, 2017

Description

generate-load-scripts has problems with casing. This doesn't really affect Windows or macOS since they don't care, but this really affects anything linux related.

paket exits with the error

Paket failed with
-> DirectoryNotFoundException: Could not find a part of the path "/app/packages/tests/Foq/lib/net45/Foq.dll".

Repro steps

https://github.com/TheAngryByrd/generate-load-scripts-casing-repro

Expected behavior

paket generates load scripts with correct casing in filepaths

Actual behavior

paket does not generate load scripts with correct casing in filepaths

Known workarounds

Use an insane OS like Windows or macOS :trollface:

@matthid

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid Jul 20, 2017

Member

Might be related to the fix of #2533

Member

matthid commented Jul 20, 2017

Might be related to the fix of #2533

@matthid matthid added the bug label Jul 20, 2017

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jul 21, 2017

Member

@TheAngryByrd are you sure it's still an issue?
Is so can you please run it with -v?

Member

forki commented Jul 21, 2017

@TheAngryByrd are you sure it's still an issue?
Is so can you please run it with -v?

@forki

This comment has been minimized.

Show comment
Hide comment
@forki
Member

forki commented Jul 21, 2017

image

@matthid

This comment has been minimized.

Show comment
Hide comment
@matthid

matthid Jul 21, 2017

Member

repro works after converting build.sh to unix line feeds on windows...
I usually use git bash and dos2unix build.sh for that.

Problem is that windows git is checking out with windows crlf and if that gets added to the container it won't work.

Member

matthid commented Jul 21, 2017

repro works after converting build.sh to unix line feeds on windows...
I usually use git bash and dos2unix build.sh for that.

Problem is that windows git is checking out with windows crlf and if that gets added to the container it won't work.

@forki

This comment has been minimized.

Show comment
Hide comment
@forki

forki Jul 21, 2017

Member

nice repro!

Member

forki commented Jul 21, 2017

nice repro!

@forki forki closed this in 3d84116 Jul 21, 2017

@TheAngryByrd

This comment has been minimized.

Show comment
Hide comment
@TheAngryByrd

TheAngryByrd Jul 21, 2017

Contributor

🎸

Contributor

TheAngryByrd commented Jul 21, 2017

🎸

@agross

This comment has been minimized.

Show comment
Hide comment
@agross

agross Jul 28, 2017

Contributor

Problem is that windows git is checking out with windows crlf

Can be solved with .gitattributes:
https://github.com/fsprojects/Paket/blob/master/completion/.gitattributes

Contributor

agross commented Jul 28, 2017

Problem is that windows git is checking out with windows crlf

Can be solved with .gitattributes:
https://github.com/fsprojects/Paket/blob/master/completion/.gitattributes

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