Skip to content

Comments

docs: Move project contributors info into the correct file.#119

Merged
crandmck merged 6 commits intomainfrom
fix-readme
Jun 17, 2025
Merged

docs: Move project contributors info into the correct file.#119
crandmck merged 6 commits intomainfrom
fix-readme

Conversation

@crandmck
Copy link
Collaborator

Other misc cleanup.

@crandmck crandmck marked this pull request as draft June 17, 2025 18:33
@crandmck crandmck changed the title Move docs for project contributors into the correct file. docs: Move project contributors info into the correct file. Jun 17, 2025
@crandmck crandmck marked this pull request as ready for review June 17, 2025 18:53
- Copies the appropriate libraries for your platform from `artifacts/` to `src/c2pa/libs/`
- Installs the package in development mode, so you can make changes to the Python code without reinstalling.

## Build from source
Copy link
Collaborator Author

Choose a reason for hiding this comment

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

Is this section still needed?

Copy link
Collaborator

Choose a reason for hiding this comment

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

No, it isn't anymore, good catch.

If you feel something should be mentioned for building everything yourself, it has been replaced by running https://github.com/contentauth/c2pa-python/blob/main/Makefile#L20 followed by https://github.com/contentauth/c2pa-python/blob/main/Makefile#L100 and then https://github.com/contentauth/c2pa-python/blob/main/Makefile#L26. But the shortcut command is mentioned in another part of the docs already, the one you're moving around.

- Copies the appropriate libraries for your platform from `artifacts/` to `src/c2pa/libs/`
- Installs the package in development mode, so you can make changes to the Python code without reinstalling.

## Build from source
Copy link
Collaborator

Choose a reason for hiding this comment

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

No, it isn't anymore, good catch.

If you feel something should be mentioned for building everything yourself, it has been replaced by running https://github.com/contentauth/c2pa-python/blob/main/Makefile#L20 followed by https://github.com/contentauth/c2pa-python/blob/main/Makefile#L100 and then https://github.com/contentauth/c2pa-python/blob/main/Makefile#L26. But the shortcut command is mentioned in another part of the docs already, the one you're moving around.

@crandmck crandmck merged commit d9cafc4 into main Jun 17, 2025
13 of 16 checks passed
@crandmck crandmck deleted the fix-readme branch June 17, 2025 20:13
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