Skip to content

Update names of environment variables as recommended by the latest CI…#540

Merged
jzmaddock merged 5 commits into
developfrom
ci_update_2026
Jul 8, 2026
Merged

Update names of environment variables as recommended by the latest CI…#540
jzmaddock merged 5 commits into
developfrom
ci_update_2026

Conversation

@jzmaddock

Copy link
Copy Markdown
Collaborator

… scripts.

@jzmaddock

Copy link
Copy Markdown
Collaborator Author

@pdimov do you happen to know if depinst is compatible with being called on Boost.Config? All the drone actions are failing with:

++ python tools/boostdep/depinst/depinst.py --include benchmark --include example --include examples --include tools config
Installing: config, headers, ../tools/boost_install, ../tools/build, ../tools/cmake
fatal: destination path '/drone/boost-root/libs/config' already exists and is not an empty directory.
fatal: clone of 'https://github.com/boostorg/config.git' into submodule path '/drone/boost-root/libs/config' failed
Failed to clone 'libs/config'. Retry scheduled
fatal: destination path '/drone/boost-root/libs/config' already exists and is not an empty directory.
fatal: clone of 'https://github.com/boostorg/config.git' into submodule path '/drone/boost-root/libs/config' failed
Failed to clone 'libs/config' a second time, aborting

Interestingly, our CI files do not call depinst anywhere, so I assume there must be a change in the drone scripts somewhere that's causing this @sdarwin ?

@sdarwin

sdarwin commented Jul 8, 2026

Copy link
Copy Markdown
Contributor

@jzmaddock please merge PR 536

@jzmaddock

Copy link
Copy Markdown
Collaborator Author

Ah, many thanks @sdarwin I knew I'd seen something relevant but not where...

@pdimov

pdimov commented Jul 8, 2026

Copy link
Copy Markdown
Member

Config is special-cased in depinst as an "essential" dependency and is always installed (it's used by the Jamfiles), which doesn't work that well when the library being depinst-alled is Config itself. :-)

I'll look into fixing this.

@jzmaddock

Copy link
Copy Markdown
Collaborator Author

I'll look into fixing this.

Thanks to Sam Darwin I have things working again, so don't worry too much!

@jzmaddock jzmaddock merged commit 8520f33 into develop Jul 8, 2026
17 of 18 checks passed
@pdimov

pdimov commented Jul 8, 2026

Copy link
Copy Markdown
Member

I fixed it anyway.

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