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

Fix errors related to executing dotnet #16

Merged
merged 2 commits into from
Sep 29, 2023
Merged

Fix errors related to executing dotnet #16

merged 2 commits into from
Sep 29, 2023

Conversation

ywt114
Copy link
Contributor

@ywt114 ywt114 commented Sep 15, 2023

The problem arises from rm rf/usr/share/dotnet. The solution is to first run apt-get update to update the software package list, and then run apt-get remove - y '^ dotnet -. *' to solve the execution error issue. I have tested and just submitted the issues, but I don't know if they will be killed. You can also fork the project and modify it yourself in the action.yml file by adding a sudo apt-get update before line 174, And there is a duplication between sudo rm - rf/usr/share/dotnet and sudo apt-get remove - y '^ dotnet -. *'. Change to sudo rm - rf/usr/share/dotnet | | true and sudo apt get remove - y '^ dotnet -. *' | | true

@normanmaurer
Copy link

Please consider merging this one... Without this PR builds will continue to fail

@thorstenwagner
Copy link

Workaround: If you set only android to true, it runs.

@subhamkrai
Copy link

Please consider this change, we are also facing issue with google-cloud pkg

@subhamkrai
Copy link

@jlumbroso @gruve-p @rake5k ^^

@gruve-p
Copy link
Contributor

gruve-p commented Sep 26, 2023

@jlumbroso @gruve-p @rake5k ^^

I am not the maintainer of this repo nor do I have any merge rights, Pinging me will not solve your request.

@jlumbroso
Copy link
Owner

@ywt114 Thanks for your PR! I will merge this, but for future reference, rather than true, I prefer a log error message, as @ChrisCarini did very beautifully here:
#8 (review)

@ChrisCarini
Copy link
Contributor

I prefer a log error message, as @ChrisCarini did very beautifully here:

Thank you for the kind words, @jlumbroso ! 😃

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.

7 participants