Skip to content
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

git package redistrubutes some Microsoft copyrighted files + git post install script partially fails + bash stdin stdout files test fails #56

Open
kiwi0fruit opened this issue Feb 19, 2019 · 11 comments

Comments

@kiwi0fruit
Copy link

kiwi0fruit commented Feb 19, 2019

  • This git package redistrubutes some Microsoft copyrighted files. This happen because of the git post install script (that should be run on user machine only!) copying some Windows files.
  • Git post install script invoked the wrong way. The file says that it should be:
     git-bash.exe --no-needs-console --hide --no-cd --command=post-install.bat
  • But fixing it doesn's fix some unknown errors in the post install script. So that conda build fails if used this way:
    git-bash.exe --no-needs-console --hide --no-cd --command=post-install.bat || exit 1
    or this way:
    call post-install.bat || exit 1

Actually this package does redistribution the wrong way. The right way is to do literally the same as original git devs intended. So I suggest go with no_link and run post-install on the user machine.

@kiwi0fruit
Copy link
Author

These issues were fixed in this conda package: https://github.com/kiwi0fruit/py-bash/tree/master/conda-forge/git-bash

@scopatz
Copy link
Member

scopatz commented Feb 25, 2019

So should we close this?

@kiwi0fruit
Copy link
Author

This package still:

  1. redistributes a few files copied from Windows that have Microsoft copyright hats.
  2. doesn't pass this test: bash -c "if [[ $(echo hello | cp /dev/stdin /dev/stdout) != 'hello' ]]; then exit 1; fi" because post install script should run on user machine not on build machine.

I'd rather see this issue #53 resolved (maybe even in favor of being the same as in main channel) as this is just you know ugly.

@scopatz
Copy link
Member

scopatz commented Feb 25, 2019

OK - @kiwi0fruit, I am unclear why (1) is a problem, as long as they have open source licenses

@kiwi0fruit
Copy link
Author

Hm, it's not clear what license they have. I don't remember them being open source. If I remember correctly they are proprietary (but that needs checking).

@scopatz
Copy link
Member

scopatz commented Feb 25, 2019

If they are modifying git, then they'd be GPL

@kiwi0fruit
Copy link
Author

They are simply files copied from C:\Windows...

@scopatz
Copy link
Member

scopatz commented Feb 25, 2019

Ahh that is not great.

@kiwi0fruit
Copy link
Author

kiwi0fruit commented Feb 25, 2019 via email

@kiwi0fruit
Copy link
Author

But there is really something minor. Like current machine hosts with copyrighted commented examples and the only trivial working line.

@kiwi0fruit
Copy link
Author

Current version of conda-forge::git still fails this test on Windows:

bash -c "if [[ $(echo hello | cp /dev/stdin /dev/stdout) != 'hello' ]]; then exit 1; fi"

While installer-installed git-bash passes it OK.

@kiwi0fruit kiwi0fruit changed the title git package redistrubutes some Microsoft copyrighted files + git post install script partially fails git package redistrubutes some Microsoft copyrighted files + git post install script partially fails + bash stdin stdout files test fails Dec 10, 2019
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants