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.

Sign up for GitHub

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

Already on GitHub? Sign in to your account

Jump to bottom

[Docs] - FuseML Windows binary missing sh and openssl #48

Closed
alfsuse opened this issue Mar 23, 2021 · 1 comment 路 Fixed by #104
Closed

[Docs] - FuseML Windows binary missing sh and openssl #48

alfsuse opened this issue Mar 23, 2021 · 1 comment 路 Fixed by #104
Labels
area/docs Improvements or additions to documentation area/installer Installation related requests good first issue Good for newcomers help wanted Extra attention is needed
Projects
Fuseml Repo FuseML Project Board

Comments

@alfsuse
Copy link
Member

alfsuse commented Mar 23, 2021

Apparently this is an "hard" requirement to run fuseML under Windows.

What is happening

When fuseml.exe is executed user receive te following error:

PS C:\Users\AlessandroFesta\Downloads> .\fuseml.exe
馃敟 Not found: sh
馃敟 Not found: openssl
Cannot operate: Please check your PATH, some of our dependencies were not found

What is expected

Use should have the fuseML help as output

##Suggested fix

Option A:
Document user strict requirements in a "Quick Start Guide"

Option B:
Remove the windows binary until next release
@alfsuse alfsuse added bug Something isn't working area/docs Improvements or additions to documentation area/core labels Mar 23, 2021
@alfsuse alfsuse added this to To do in Fuseml Repo via automation Mar 23, 2021
@alfsuse
Copy link
Member Author

alfsuse commented Mar 23, 2021

As further information, while fuseml may be interesting as windows binary is also true that the user may run the Linux binary under WSL to avoid the above issue.
An alternative is to offer user to consume the binary under WSL and point to their local development K8s (that anyway would run under Linux containers so is accessible from within WSL)

@alfsuse alfsuse mentioned this issue Mar 25, 2021
Closed
@alfsuse alfsuse added good first issue Good for newcomers help wanted Extra attention is needed and removed bug Something isn't working area/core labels Mar 25, 2021
@alfsuse alfsuse changed the title FuseML Windows binary missing sh and openssl [Docs] - FuseML Windows binary missing sh and openssl Mar 25, 2021
@stefannica stefannica added the area/installer Installation related requests label Apr 23, 2021
@stefannica stefannica added this to Backlog in FuseML Project Board via automation Apr 23, 2021
jsuchome added a commit to jsuchome/fuseml that referenced this issue May 25, 2021
@jsuchome
fix: Remove dependency on 'sh' and 'openssl'
6e13ef0 
These does not seem to be used at all. And without them, it should
be easier to use windows binary.

Closes fuseml#48
@jsuchome jsuchome mentioned this issue May 25, 2021
Merged
@jsuchome jsuchome moved this from Backlog to In progress in FuseML Project Board May 25, 2021
FuseML Project Board automation moved this from In progress to Done May 27, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
area/docs Improvements or additions to documentation area/installer Installation related requests good first issue Good for newcomers help wanted Extra attention is needed
Projects
FuseML Project Board
  
Done
Fuseml Repo
  
To do
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Remove dependency on 'sh' and 'openssl' jsuchome/fuseml
2 participants
@stefannica @alfsuse