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

app-emulation/wine-vanilla: fix a crash in install phase #6407

Closed
wants to merge 1 commit into
base: master
from

Conversation

Projects
None yet
5 participants
@shimataro
Contributor

shimataro commented Dec 3, 2017

Hello!

app-emulation/wine-vanilla crashes in install phase with following message:

/var/tmp/portage/app-emulation/wine-vanilla-2.0.2/temp/environment: line 3644: no match: /var/tmp/portage/app-emulation/wine-vanilla-2.0.2/image/usr/share/wine-vanilla-2.0.2/man/pl*

this pull-request will fix it.

@gentoo-repo-qa-bot

This comment has been minimized.

Show comment
Hide comment
@gentoo-repo-qa-bot

gentoo-repo-qa-bot Dec 3, 2017

Collaborator

Pull Request assignment

Areas affected: ebuilds
Packages affected: app-emulation/wine-vanilla

app-emulation/wine-vanilla: @gentoo/wine

No bug reference found in commit messages. If you would like to reference bugs in the pull request, please make sure to use GLEP 66 tags in the commit message.

If there is no bug open for this pull request, please open one. Otherwise, please link it to the commit messages as noted above.

Collaborator

gentoo-repo-qa-bot commented Dec 3, 2017

Pull Request assignment

Areas affected: ebuilds
Packages affected: app-emulation/wine-vanilla

app-emulation/wine-vanilla: @gentoo/wine

No bug reference found in commit messages. If you would like to reference bugs in the pull request, please make sure to use GLEP 66 tags in the commit message.

If there is no bug open for this pull request, please open one. Otherwise, please link it to the commit messages as noted above.

@austin987

Lgtm. I can't merge into Monday, but someone may beat me to it ;)

@gentoo-bot gentoo-bot closed this in 5cccc43 Dec 3, 2017

@NP-Hardass

This comment has been minimized.

Show comment
Hide comment
@NP-Hardass

NP-Hardass Dec 4, 2017

Contributor

@a17r This really shouldn't have been merged for just wine-vanilla. Should have been applied to all of wine-* and a patch prepared for the overlay...

Did you clear this with Austin, or take it upon yourself to merge it?

Contributor

NP-Hardass commented Dec 4, 2017

@a17r This really shouldn't have been merged for just wine-vanilla. Should have been applied to all of wine-* and a patch prepared for the overlay...

Did you clear this with Austin, or take it upon yourself to merge it?

@NP-Hardass NP-Hardass reopened this Dec 4, 2017

@NP-Hardass

This comment has been minimized.

Show comment
Hide comment
@NP-Hardass

NP-Hardass Dec 4, 2017

Contributor

@shimataro Thanks for the PR.

Contributor

NP-Hardass commented Dec 4, 2017

@shimataro Thanks for the PR.

@NP-Hardass NP-Hardass closed this Dec 4, 2017

gentoo-repo-qa-bot pushed a commit to gentoo-mirror/wine that referenced this pull request Dec 4, 2017

app-emulation/wine-any: Add force to rm of linguas
Closes: https://bugs.gentoo.org/617864
Closes: gentoo/gentoo#6407
Package-Manager: Portage-2.3.14, Repoman-2.3.6
@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Dec 7, 2017

Um, I'm experiencing this bug supposedly.
/big/volatile/tmp/portage/app-emulation/wine-vanilla-2.0.3/temp/environment: line 3608: no match: /big/volatile/tmp/portage/app-emulation/wine-vanilla-2.0.3/image/usr/share/wine-vanilla-2.0.3/man/pl*

But I see rm -rf in there. What's wrong then?

ghost commented Dec 7, 2017

Um, I'm experiencing this bug supposedly.
/big/volatile/tmp/portage/app-emulation/wine-vanilla-2.0.3/temp/environment: line 3608: no match: /big/volatile/tmp/portage/app-emulation/wine-vanilla-2.0.3/image/usr/share/wine-vanilla-2.0.3/man/pl*

But I see rm -rf in there. What's wrong then?

@Chiitoo

This comment has been minimized.

Show comment
Hide comment
@Chiitoo
Contributor

Chiitoo commented Dec 7, 2017

@ghost

This comment has been minimized.

Show comment
Hide comment
@ghost

ghost Dec 7, 2017

@Chiitoo, oh, silly me. I see, globbing, complicated:) I'll make it install locally for this time and hopefully forget until next update.

ghost commented Dec 7, 2017

@Chiitoo, oh, silly me. I see, globbing, complicated:) I'll make it install locally for this time and hopefully forget until next update.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment