synapse icon indicating copy to clipboard operation
synapse copied to clipboard

Allow Docker image to be built on Windows

Open turt2live opened this issue 1 year ago • 5 comments

Sometimes (often) Git forgets to CRLF->LF and this breaks execution of /start.py because #!/usr/local/bin/python\r can't be found (noting carriage return).

turt2live avatar Jul 09 '24 16:07 turt2live

maybe we should add a .gitattributes of * text eol=lf instead?

Changing the dockerfile kind of feels like the wrong level for this, and what limit does it stop at? :S

reivilibre avatar Jul 16 '24 11:07 reivilibre

I'm not sure the gitattributes would be enough to make Git for Windows happy, but can certainly give it a try on my next Synapse PR :)

turt2live avatar Jul 16 '24 15:07 turt2live

I'm not sure the gitattributes would be enough to make Git for Windows happy, but can certainly give it a try on my next Synapse PR :)

By this, do you mean that you're going to add a .gitattributes file in your next PR, or that you're hoping we add one?

anoadragon453 avatar Jul 17 '24 14:07 anoadragon453

Taking this out of the review queue for now while the alternative is evaluated.

anoadragon453 avatar Jul 17 '24 14:07 anoadragon453

I'm not sure the gitattributes would be enough to make Git for Windows happy, but can certainly give it a try on my next Synapse PR :)

By this, do you mean that you're going to add a .gitattributes file in your next PR, or that you're hoping we add one?

sorry, missed this: I was hoping to give it a try the next time I touch Synapse. This PR can sit until then, or be closed - whichever the team prefers.

turt2live avatar Aug 07 '24 16:08 turt2live

I'm closing this pending me actually trying .gitattributes

turt2live avatar Jun 24 '25 15:06 turt2live