Skip to content

Convert to rattler#226

Merged
apmorton merged 14 commits intoconda-forge:mainfrom
apmorton:convert_feedstock_to_v1_recipe_format
Sep 1, 2025
Merged

Convert to rattler#226
apmorton merged 14 commits intoconda-forge:mainfrom
apmorton:convert_feedstock_to_v1_recipe_format

Conversation

@apmorton
Copy link
Contributor

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

@conda-forge-admin
Copy link
Contributor

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe/recipe.yaml) and found it was in an excellent condition.

@apmorton
Copy link
Contributor Author

@mattip decided to do this while iterating locally on 2.49

pixi/rattler are a lot faster, so local builds actually get to the ray build much quicker.

@apmorton
Copy link
Contributor Author

No idea whats up with FATAL: corrupt installation: file '$BUILD_PREFIX/share/bazel/install/ee1819c7c6ef276537e1ca6a5f696249/A-server.jar' is missing or modified. Please remove '$BUILD_PREFIX/share/bazel/install/ee1819c7c6ef276537e1ca6a5f696249' and try again., but I also get this locally for linux builds.

Very odd that non-linux builds are working just fine.

+ ]
+ )
+
+- SOURCEFORGE_MIRRORS = ["cfhcable", "superb-sea2", "cytranet", "iweb", "gigenet", "ayera", "astuteinternet", "pilotfiber", "svwh"]
Copy link
Contributor Author

Choose a reason for hiding this comment

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

many of these mirrors are dead, and it causes the build to hang for several minutes while timing out before trying the next mirror.

Copy link
Contributor

Choose a reason for hiding this comment

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

Nice

Copy link
Contributor

Choose a reason for hiding this comment

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

Note lzma is also available on github

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I went looking pretty hard for xz-5.2.7.tar.gz with hash 06327c2ddc81e126a6d9a78b0be5014b976a2c0832f492dcfc4755d7facf6d33 from a better mirror and gave up after a while; I'm hoping this is short lived as I intend to start devendoring some things (starting with boost as a proof of concept)

Copy link
Contributor

Choose a reason for hiding this comment

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

I intend to start devendoring some things

Whohoo!

I had a dream we removed bazel and use meson-python to build _raylet.so, core/src/ray/raylet/raylet, core/src/ray/gcs/gcs_server.

@apmorton
Copy link
Contributor Author

apmorton commented Sep 1, 2025

@mattip I believe all the current build will pass momentarily - objections to landing this first and rebasing the 2.49 branch?

linux builds are nearly 30 minutes faster with this.

@mattip
Copy link
Contributor

mattip commented Sep 1, 2025

Gr8! Speedups are welcome.

@mattip mattip added automerge Merge the PR when CI passes and removed automerge Merge the PR when CI passes labels Sep 1, 2025
@mattip
Copy link
Contributor

mattip commented Sep 1, 2025

Did you bump the build number?

@apmorton
Copy link
Contributor Author

apmorton commented Sep 1, 2025

yes; bumped to 2

please don't turn on automerge; once builds are green I was going to remove @krfricke, rerender, and merge.

- {{ compiler('cxx') }}
host:
- python
number: 2
Copy link
Contributor

Choose a reason for hiding this comment

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

Ahh, the file name changed ✅

@mattip
Copy link
Contributor

mattip commented Sep 1, 2025

Can you also try removing these 2 lines from the windows build?

echo build --jobs=1 >> ..\.bazelrc
echo build --subcommands=pretty_print >> ..\.bazelrc

@apmorton apmorton merged commit 32d6d7f into conda-forge:main Sep 1, 2025
4 of 7 checks passed
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.

3 participants