Skip to content

Issue 6282 - BUG - out of tree build fails#6283

Merged
Firstyear merged 1 commit into
389ds:mainfrom
Firstyear:20240801-minor-build-fixes
Aug 1, 2024
Merged

Issue 6282 - BUG - out of tree build fails#6283
Firstyear merged 1 commit into
389ds:mainfrom
Firstyear:20240801-minor-build-fixes

Conversation

@Firstyear

Copy link
Copy Markdown
Contributor

Bug Description: When building with an external builddir the build fails.

Fix Description: Due to how python setup.py works, we have to specifically template it into srcdir rather than the build dir.

Rename cargo/config to cargo/config.toml to avoid future build errors.

fixes: #6282

Author: William Brown william@blackhats.net.au

Review by: ???

Bug Description: When building with an external builddir
the build fails.

Fix Description: Due to how python setup.py works, we have
to specifically template it into srcdir rather than the
build dir.

Rename cargo/config to cargo/config.toml to avoid future
build errors.

fixes: 389ds#6282

Author: William Brown <william@blackhats.net.au>

Review by: ???

@vashirov vashirov left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

LGTM

@Firstyear Firstyear merged commit ea3cc55 into 389ds:main Aug 1, 2024
@Firstyear Firstyear deleted the 20240801-minor-build-fixes branch August 1, 2024 23:25
@Firstyear

Copy link
Copy Markdown
Contributor Author

@vashirov Any issues with this in 2.5? Seems like we need it there for SUSE.

@vashirov

vashirov commented Aug 2, 2024

Copy link
Copy Markdown
Member

No issues, should be fine.

Firstyear added a commit that referenced this pull request Aug 2, 2024
Bug Description: When building with an external builddir
the build fails.

Fix Description: Due to how python setup.py works, we have
to specifically template it into srcdir rather than the
build dir.

Rename cargo/config to cargo/config.toml to avoid future
build errors.

fixes: #6282

Author: William Brown <william@blackhats.net.au>

Review by: @vashirov (Thanks!)
@Firstyear

Copy link
Copy Markdown
Contributor Author
To github.com:389ds/389-ds-base.git
   a7deea8b3..161888969  389-ds-base-2.5 -> 389-ds-base-2.5

Thanks!

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.

Out of tree builds fail

2 participants