Skip to content

Conversation

@trivialfis
Copy link
Member

@trivialfis trivialfis commented Feb 26, 2025

  • Generate the variants instead of using patches.
  • Update links to s3 buckets.
  • Build the glibc warning into XGBoost instead of using a patch. This might affect others who build from source but it should be fine since we don't want to support old glibc anyway.

I'm using simple string match and replace to generate files from templates. But we can switch to jinja if necessary (when things are more complicated than we can handle, say, many cuda versions).

In the long term, I would like to gather these scripts into a manager project.

todo

  • Need to check the wheels with twine check.
  • xgboost-cpu is missing for win64.
  • Is it expected that the PyPI tag macosx_10_15_x86_64.macosx_11_0_x86_64.macosx_12_0_x86_64 is now simply macosx_10_15_x86_64?

@trivialfis
Copy link
Member Author

@hcho3 Could you please help take a look when you are available? I'm not sure how to proceed with the win64 xgboost-cpu along with the changed macos wheel.

@trivialfis trivialfis requested a review from hcho3 February 26, 2025 14:27
@trivialfis trivialfis changed the base branch from release_3.0.0 to master February 26, 2025 15:17
@hcho3 hcho3 changed the title [wip] Update release scripts [skip ci] [wip] Update release scripts Feb 26, 2025
IN_PATH = os.path.join(PY_PACKAGE, "pyproject.toml.in")
OUT_PATH = os.path.join(PY_PACKAGE, "pyproject.toml")

WHL_CPU = """
Copy link
Member Author

Choose a reason for hiding this comment

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

We can probably add this to the toml file unconditionally.

@hcho3
Copy link
Collaborator

hcho3 commented Feb 26, 2025

Is it expected that the PyPI tag macosx_10_15_x86_64.macosx_11_0_x86_64.macosx_12_0_x86_64 is now simply macosx_10_15_x86_64?

Yes, the new tag should be sufficient. I added a test to ensure that the wheel can be installed into latest MacOS.

I also added a CI pipeline to build xgboost-cpu on Windows.

@trivialfis trivialfis merged commit 8bd70dd into dmlc:master Feb 27, 2025
59 of 60 checks passed
@trivialfis trivialfis deleted the xgboost-cpu-build branch February 27, 2025 07:45
@trivialfis trivialfis mentioned this pull request Feb 27, 2025
6 tasks
@trivialfis trivialfis changed the title [wip] Update release scripts Update release scripts Feb 27, 2025
trivialfis added a commit to trivialfis/xgboost that referenced this pull request Feb 27, 2025
trivialfis added a commit to trivialfis/xgboost that referenced this pull request Feb 28, 2025
- Generate the variants instead of using patches.
- Update links to s3 buckets.
- Build the glibc warning into XGBoost instead of using a patch
- Build xgboost-cpu on Windows

---------

Co-authored-by: Hyunsu Cho <[email protected]>
trivialfis added a commit that referenced this pull request Feb 28, 2025
- Generate the variants instead of using patches.
- Update links to s3 buckets.
- Build the glibc warning into XGBoost instead of using a patch
- Build xgboost-cpu on Windows

---------

Co-authored-by: Hyunsu Cho <[email protected]>
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.

2 participants