Skip to content

new build, test and release workflow; makefile refactor; fixes to guarantee cross-platform compatibility#1

Merged
marcobambini merged 105 commits intomainfrom
update-build-pipeline
May 22, 2025
Merged

new build, test and release workflow; makefile refactor; fixes to guarantee cross-platform compatibility#1
marcobambini merged 105 commits intomainfrom
update-build-pipeline

Conversation

@Gioee
Copy link
Member

@Gioee Gioee commented May 22, 2025

No description provided.

Gioee added 30 commits May 16, 2025 09:47
…le on some platforms like Android and Windows. memset achieves the same functionality, is standardized, and ensures broader compatibility across systems.
@Gioee Gioee requested review from andinux and marcobambini May 22, 2025 07:25
@Gioee Gioee self-assigned this May 22, 2025
@Gioee Gioee added the enhancement New feature or request label May 22, 2025
@Gioee Gioee changed the title Update build pipeline new build, test and release workflow; makefile refactor; fixes to guarantee cross-platform compatibility May 22, 2025
Copy link
Member Author

Choose a reason for hiding this comment

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

updated workflow with sqlite-sync build, test and release; the release process (which includes coverage website update) is made only in the main branch; build and tests are done also in other dev branches

Copy link
Member Author

Choose a reason for hiding this comment

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

Since object files need to be built with different flags now makefile builds loadable extension object files inside build/release and unit test object files in build/test, they're then linked in the dist folder.

I've also changed variables names to easily understand them. In make test now coverage is an option, it's not always on.

Added windows def file gen and help message.

All object files are now built by the same pattern rule (the only exception is sqlite which uses different cflags)

Copy link
Member Author

Choose a reason for hiding this comment

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

Added test coverage badge

Copy link
Member Author

Choose a reason for hiding this comment

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

since the network folder now had only one folder inside I've removed one unused folder level.

Copy link
Member Author

Choose a reason for hiding this comment

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

I've just moved this file, it's wrongly marked as deleted

Copy link
Member Author

Choose a reason for hiding this comment

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

removed macOS compiled libcurl that was only used for development

Copy link
Member Author

Choose a reason for hiding this comment

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

removed unused curl test

Copy link
Member Author

Choose a reason for hiding this comment

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

I've removed this duplicate of what was inside the network folder

Copy link
Member Author

Choose a reason for hiding this comment

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

I've made a small change on how test functions are executed in the main to return an error number in case of errors.

if (clone) cloudsync_memory_free(clone);

if (rc != SQLITE_DONE) {
if (rc != SQLITE_OK) {
Copy link
Member Author

Choose a reason for hiding this comment

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

If I'm not wrong rc is always different from SQLITE_DONE because of what's on line 2034

@Gioee Gioee marked this pull request as ready for review May 22, 2025 07:59
@marcobambini marcobambini merged commit 3543e6b into main May 22, 2025
9 checks passed
@Gioee Gioee deleted the update-build-pipeline branch June 13, 2025 18:34
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

enhancement New feature or request

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants