Skip to content

Conversation

@sfc-gh-abozkurt
Copy link
Collaborator

@sfc-gh-abozkurt sfc-gh-abozkurt commented Nov 4, 2025

  • Remove fast and ci makefile targets. Instead, make the default build target smart to skip building duckdb, if already built before.
  • Fix broken avro build dependency on make targets.

Makefile Outdated
$(MAKE) -C pg_lake_benchmark install

duckdb_pglake: submodules
# skip duckdb_pglake build if duckdb_pglake library already exists in PG_LIBDIR
Copy link
Collaborator

Choose a reason for hiding this comment

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

what if I'm checking out a branch that makes changes to duckdb_pglake?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

what about passing a make variable FORCE_DUCKDB_REBUILD in these cases?

Copy link
Collaborator

Choose a reason for hiding this comment

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

What about something like make fast for the no rebuild case? :)

(Or make full or something for rebuild)

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