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

dev-python/pyxdg: Remove unused patch #300

Merged
merged 1 commit into from
Oct 28, 2015
Merged

Conversation

anthonyryan1
Copy link
Contributor

files/pyxdg-subprocess.patch is no longer used

Fixes https://bugs.gentoo.org/540940

@gentoo-repo-qa-bot
Copy link
Collaborator

😞 The QA check for this pull request has found the following issues:

https://qa-reports.gentoo.org/output/gentoo-ci/ebee082/5.html#l2568

Please note that the issues may come from the underlying Gentoo repository state rather than the pull request itself.

@mgorny
Copy link
Member

mgorny commented Oct 27, 2015

Please don't mix metadata change with patch removal. Those are two irrelevant changes which deserve separate commits. This would be especially important if we ever needed to restore the patch.

@anthonyryan1
Copy link
Contributor Author

@mgorny Commit updated to only remove the patch file. I'll send a separate pull request for the metadata changes at a later time.

files/pyxdg-subprocess.patch is no longer used

Fixes https://bugs.gentoo.org/540940
@gentoo-repo-qa-bot
Copy link
Collaborator

😞 The QA check for this pull request has found the following issues:

https://qa-reports.gentoo.org/output/gentoo-ci/a8fdbb9/5.html#l2273

Please note that the issues may come from the underlying Gentoo repository state rather than the pull request itself.

@gentoo-bot gentoo-bot merged commit ee4570c into gentoo:master Oct 28, 2015
mgorny added a commit that referenced this pull request Oct 28, 2015
@mgorny
Copy link
Member

mgorny commented Oct 28, 2015

Done, thanks. Next time, please consider signing the commit.

@mgorny
Copy link
Member

mgorny commented Oct 28, 2015

As in, git rebase -S probably ;-).

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
4 participants