Skip to content

Conversation

@CD3
Copy link

@CD3 CD3 commented Mar 29, 2025

I've implemented an index command for managing local conan recipe indexes. This makes it easy to add conan remotes from git repositories.

For example:

$ cppship index add --name cd3 --git https://github.com/CD3/cd3-conan-packages

This will clone the git repository to ~/.cppship/index_store/cd3 and add that directory as a local package index so you can use the recipes in the index as conan dependencies.

I also updated the install command. It started as a fix to create the destination directory and all parents so you don't get an error if you try to install to a directory that does not exist, and then I added a --root option and changed to default install location to ~/.cppship/bin to reflect cargo.

CD3 added 4 commits March 23, 2025 03:34
--root can be used to specify the install root.
The create_if_not_exist(...) function will now create parent directories
if they do not exists.

Added call to create_if_not_exists(...) when installing executable.
…al recipe index repos

New `index` command can be used to add local recipe index repos to the
.cppship directory. This is to make it easy add conan remotes from local
directories or git repos.
@qqiangwu
Copy link
Owner

Awesome.

Can you please split the pr into two, one for install command and one for index command so that I can review it easier?

@qqiangwu
Copy link
Owner

In addition, remove the publish command for the moment.

@CD3
Copy link
Author

CD3 commented Mar 29, 2025

Yes, I was going to do that but ran out of time this morning. I'll split it into the install command update and the index command addition on top of that (it depends on the function added to support the install update).

I removed the publish command from main but forgot to clean up the other files.

Probably won't be able to do anything for a few day.

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