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/watchfiles: Keyword 0.16.1 sparc, #852878 #26739

Closed
wants to merge 5 commits into from

Conversation

matoro
Copy link
Contributor

@matoro matoro commented Aug 4, 2022

No description provided.

Signed-off-by: matoro <matoro@users.noreply.github.com>
Signed-off-by: matoro <matoro@users.noreply.github.com>
Signed-off-by: matoro <matoro@users.noreply.github.com>
@gentoo-bot
Copy link

Pull Request assignment

Submitter: @matoro
Areas affected: ebuilds
Packages affected: app-text/mdbook, dev-python/boltons, dev-python/dirty-equals, dev-python/watchfiles, dev-util/maturin

app-text/mdbook: @ionenwks
dev-python/boltons: @ionenwks
dev-python/dirty-equals: @gentoo/python
dev-python/watchfiles: @gentoo/python
dev-util/maturin: @ionenwks, @steils, @gentoo/rust

Linked bugs

No bugs to link found. If your pull request references any of the Gentoo bug reports, please add appropriate GLEP 66 tags to the commit message and request reassignment.

If you do not receive any reply to this pull request, please open or link a bug to attract the attention of maintainers.


In order to force reassignment and/or bug reference scan, please append [please reassign] to the pull request title.

Docs: Code of ConductCopyright policy (expl.) ● DevmanualGitHub PRsProxy-maint guide

@gentoo-bot gentoo-bot added assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits. labels Aug 4, 2022
@@ -343,7 +343,7 @@ LICENSE="
BSD CC0-1.0 ISC MIT MPL-2.0 openssl unicode
doc? ( CC-BY-4.0 OFL-1.1 )"
SLOT="0"
KEYWORDS="~amd64 ~arm ~arm64 ~riscv ~x86"
KEYWORDS="~amd64 ~arm ~arm64 ~riscv ~sparc ~x86"
Copy link
Contributor

Choose a reason for hiding this comment

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

Does this really work? Sparc fixes are in maturin-0.13.2_beta1 (okay to keyword that one instead)

Copy link
Contributor Author

Choose a reason for hiding this comment

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

Hmm, it seems it did install beta1, but nattka apply chose to add the kw to this version instead. How should I have invoked it instead? I used nattka apply -a sparc 852878.

Copy link
Contributor

Choose a reason for hiding this comment

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

(theory fixes anyway, upstream didn't get to test it afaik)

Copy link
Contributor

Choose a reason for hiding this comment

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

Ah okay, I believe it just doesn't try to keyword unkeyworded ebuilds given these are often live ebuilds. Probably should just do it manually.

Copy link
Contributor Author

Choose a reason for hiding this comment

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

It passes tests on real hardware. Used rust-bin-1.62.1.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice, I'll drop a quick note upstream that it's fine.

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-08-04 19:16 UTC
Newest commit scanned: 3628013
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/4c1f361f17/output.html

Signed-off-by: matoro <matoro@users.noreply.github.com>
Signed-off-by: matoro <matoro@users.noreply.github.com>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2022-08-04 19:51 UTC
Newest commit scanned: 3362c67
Status: ✅ good

There are existing issues already. Please look into the report to make sure none of them affect the packages in question:
https://qa-reports.gentoo.org/output/gentoo-ci/be08d01c57/output.html

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
assigned PR successfully assigned to the package maintainer(s). no bug found No Bug/Closes found in the commits.
Projects
None yet
4 participants