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

Add Rust 1.78.0 #36710

Closed
wants to merge 8 commits into from
Closed

Add Rust 1.78.0 #36710

wants to merge 8 commits into from

Conversation

navi-desu
Copy link
Contributor

spidermonkey/firefox fail to build, seemingly due to nightly features that got removed?

built with and without system-llvm, ripgrep and cbindgen built fine
it also bootstrapped itself fine iirc (will double check tomorrow as well)

missing building with FEATURES=test, will run those tomorrow

don't have any non-x86_64 machines to test that end


Please check all the boxes that apply:

  • I can submit this contribution in agreement with the Copyright Policy.
  • I have certified the above via adding a Signed-off-by line to every commit in the pull request.
  • This contribution has not been created with the assistance of Natural Language Processing artificial intelligence tools, in accordance with the AI policy.
  • I have run pkgcheck scan --commits --net to check for issues with my commits.

Please note that all boxes must be checked for the pull request to be merged.

@gentoo-bot
Copy link

Pull Request assignment

Submitter: @navi-desu
Areas affected: ebuilds
Packages affected: dev-lang/rust, dev-lang/rust-bin, sys-devel/rust-std, virtual/rust

dev-lang/rust: @bowlofeggs, @navi-desu, matoro_gentoo[at]matoro.tk, @gentoo/rust
dev-lang/rust-bin: @bowlofeggs, @navi-desu, matoro_gentoo[at]matoro.tk, @gentoo/rust
sys-devel/rust-std: @bowlofeggs, @navi-desu, matoro_gentoo[at]matoro.tk, @gentoo/rust
virtual/rust: @bowlofeggs, @navi-desu, matoro_gentoo[at]matoro.tk, @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.


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 self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else) assigned PR successfully assigned to the package maintainer(s). labels May 17, 2024
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-17 04:40 UTC
Newest commit scanned: 58ce821
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/d0b7279cf9/output.html

@stefson
Copy link
Contributor

stefson commented May 17, 2024

I think you forgot to upgrade to llvm-18 in virtual/rust and in LLVM_VALID_SLOTS?

@stefson
Copy link
Contributor

stefson commented May 17, 2024

Also the dist useflag doesn't work anymore as it was intended, since the archive files are no longer generated. More info here: rust-lang/rust#124528

@navi-desu
Copy link
Contributor Author

fixed those issues, but noticed i was missing the wasm flag

with wasm, it's failing to build with cargo:warning=clang: error: unsupported option '-march=' for target 'wasm32-unknown-unknown' if -march flag is set, otherwise builds fine

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-18 01:40 UTC
Newest commit scanned: 43f5b3a
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/60ce44fbdf/output.html

@thesamesam
Copy link
Member

cc @juippis

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-20 01:05 UTC
Newest commit scanned: 3b5a297
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/f6a3c13b66/output.html

@juippis
Copy link
Member

juippis commented May 21, 2024

spidermonkey/firefox fail to build, seemingly due to nightly features that got removed?

I think you forgot to upgrade to llvm-18 in virtual/rust and in LLVM_VALID_SLOTS?

Probably related, Mozilla ebuilds should force a known working rust implementation to be used.

fsvm88 added a commit to fsvm88/fsvm88-overlay that referenced this pull request May 24, 2024
…refox for LLVM 18, same with bcachefstools
@fsvm88
Copy link

fsvm88 commented May 24, 2024

I was able to fix the wasm compilation issue by filtering the flags in src_configure, after line 506 ->

506     if use wasm; then
507         # CFLAGS_wasm32_unknown_unknown
508         wasm_target="wasm32-unknown-unknown"
509         export CFLAGS_${wasm_target//-/_}="$(filter-flags '-mcpu*' '-march*' '-mtune*'; echo $CFLAGS)"

The code uses the subshell to leave *FLAGS unaltered in the src_configure scope, currently crossdev code does it on its own later, just being safe in case other code is added in between.

@juippis
Copy link
Member

juippis commented May 28, 2024

spidermonkey/firefox fail to build, seemingly due to nightly features that got removed?

I think you forgot to upgrade to llvm-18 in virtual/rust and in LLVM_VALID_SLOTS?

Probably related, Mozilla ebuilds should force a known working rust implementation to be used.

Oh, but I did realize it only applies with +clang use flag. Hmm.

@juippis
Copy link
Member

juippis commented May 28, 2024

If this is only waiting for firefox, thunderbird and spidermonkey, commit the following to restrict the versions temporarily:

	virtual/pkgconfig
-	!clang? ( >=virtual/rust-1.74 )
-	!elibc_glibc? ( dev-lang/rust )
	amd64? ( >=dev-lang/nasm-2.14 )
	
	virtual/pkgconfig
+	!clang? (
+            >=virtual/rust-1.74
+            <virtual/rust-1.78
+        )
+	!elibc_glibc? ( <dev-lang/rust-1.78 )
	amd64? ( >=dev-lang/nasm-2.14 )

@navi-desu
Copy link
Contributor Author

added the wasm fix, will for firefox et all tomorrow first thing

tho, just to be sure, do i add it just for the latest (stable | testing) ebuilds of each?

@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-29 00:10 UTC
Newest commit scanned: a69fbfe
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/3cf1c38eeb/output.html

@juippis
Copy link
Member

juippis commented May 29, 2024

tho, just to be sure, do i add it just for the latest (stable | testing) ebuilds of each?

Yeah,

  • dev-lang/spidermonkey-115.11.0
  • mail-client/thunderbird-115.11.0
  • www-client/firefox-115.11.0
  • www-client/firefox-126.0.1

Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
Signed-off-by: Anna (navi) Figueiredo Gomes <navi@vlhl.dev>
@gentoo-repo-qa-bot
Copy link
Collaborator

Pull request CI report

Report generated at: 2024-05-29 21:45 UTC
Newest commit scanned: f53fd88
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/bd685fe5be/output.html

Comment on lines -117 to 126
!clang? ( >=virtual/rust-1.65 )
!clang? (
>=virtual/rust-1.65
<virtual/rust-1.78
)
!elibc_glibc? (
|| (
dev-lang/rust
<dev-lang/rust-1.78
<dev-lang/rust-bin-1.73
)
)
Copy link
Contributor

Choose a reason for hiding this comment

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

Hello everyone,

Could someone clarify why this mix of restrictions might be used for !elibc_glibc??
I mean, why not use virtual/rust instead of dev-lang/rust{,-bin} like for !clang? and why rust-bin is limited to <1.73? Similar questions are relevant for other ebuilds.

Thanks!

Copy link
Member

Choose a reason for hiding this comment

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

You can check git commit history but rust-bin is broken (for reasons unknown, someone needs to investigate it) with musl with newer versions for the purposes of building FF.

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). self-maintained The PR changes only packages that are maintained by the submitter (i.e. no need to ask anybody else)
Projects
None yet
8 participants