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

(#2238) Mono build don't hardcode xbuild path #2239

Merged
merged 1 commit into from
Apr 21, 2021

Conversation

TheCakeIsNaOH
Copy link
Member

Removes basedir from Mono build on non-windows platforms.
This lets xbuild be located at other locations, as long at it's on the path

Fixes: #2238

I'm not %100 sure that this is the best solution, but it seems to work everwhere I have tested it.
Ubuntu, Debian, Docker, and MacOS (in Github actions) all build.

I'm not super familiar with uppercut, so if there is a better way to do this, I'm all ears.

@gep13
Copy link
Member

gep13 commented Apr 17, 2021

@ferventcoder would you be in a position to provide some guidance here?

The idea behind this is that we would like to build Chocolatey on GitHub Actions, running on Windows, Mac and Linux, and there is an issue when trying this due to the location of the xbuild application on some runners.

I know that directly editing the UppercuT files is not recommended, so I am curious as to what the best course of action is here.

Removes basedir from Mono build on non-windows platforms.
This lets xbuild be located at other locations, as long at it's on
the path
Copy link
Member

@gep13 gep13 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

LGTM!

@gep13
Copy link
Member

gep13 commented Apr 21, 2021

@TheCakeIsNaOH thanks for investigating this, and for finding a solution. Will get this merged once the CI builds complete.

@gep13
Copy link
Member

gep13 commented Apr 21, 2021

There is an open GitHub Actions incident which I believe is why the Windows build isn't working correctly:

https://www.githubstatus.com/incidents/cj7gzzj30411

Since this change only impacts the Posix build, which has passed, I am going to go ahead and merge this in.

@gep13 gep13 merged commit 97571f6 into chocolatey:stable Apr 21, 2021
@gep13
Copy link
Member

gep13 commented Apr 21, 2021

@TheCakeIsNaOH thanks again for getting this fixed up!

@gep13 gep13 linked an issue Apr 21, 2021 that may be closed by this pull request
gep13 added a commit that referenced this pull request Apr 21, 2021
Now that this PR has been merged in:

#2239

It should be possible to add Mac build into the main GitHub Action
workflow.

Co-authored-by: TheCakeIsNaOH <[email protected]>
@TheCakeIsNaOH TheCakeIsNaOH deleted the xbuild-path branch September 22, 2021 14:01
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

Successfully merging this pull request may close these issues.

Mono build on non-windows cannot find xbuild if it is not in /usr/bin
2 participants