Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

debug tidy + unix sock via files pairs #257

Merged
merged 29 commits into from
Aug 28, 2024
Merged

debug tidy + unix sock via files pairs #257

merged 29 commits into from
Aug 28, 2024

Conversation

pmp-p
Copy link
Collaborator

@pmp-p pmp-p commented Aug 25, 2024

DEBUG=false for cibuild should close #256

unix socket emulation will allow to complety build without socket support and use C stdlib (native/wasi/emscripten) filesystem support as a unified replacement

@pmp-p pmp-p marked this pull request as ready for review August 25, 2024 11:03
@pmp-p pmp-p marked this pull request as draft August 26, 2024 07:57
Copy link
Contributor

github-actions bot commented Aug 26, 2024

@pmp-p
Copy link
Collaborator Author

pmp-p commented Aug 26, 2024

was ok with debug on, retrying with off or switching to 3.1.65final

@pmp-p pmp-p marked this pull request as ready for review August 27, 2024 13:05
@pmp-p pmp-p requested review from msfstef and samwillis August 27, 2024 13:05
@pmp-p
Copy link
Collaborator Author

pmp-p commented Aug 27, 2024

also fix PGVERSION => PG_VERSION. Also will support different debug level for extension / core (CDEBUG) ( by include files and search directories for include paths ). And also allow to tune linker of core elements separately ( LDEBUG )

Copy link
Collaborator

@msfstef msfstef left a comment

Choose a reason for hiding this comment

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

Looks good! I see the docker image for the builder was also created, did you perhaps check that it works (top level of the repo run pnpm wasm:build)? Cause I know that the SDK building might fail without an exit code 1 so I'm worried about a dud docker image being uploaded (especially for ARM)

@@ -348,7 +371,7 @@ then
pushd pgvector
# path for wasm-shared already set to (pwd:pg build dir)/bin
# OPTFLAGS="" turns off arch optim (sse/neon).
PG_CONFIG=${PGROOT}/bin/pg_config emmake make OPTFLAGS="" install || exit 276
Copy link
Collaborator

Choose a reason for hiding this comment

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

do these exit codes have any meaning or are they for debugging?

Copy link
Collaborator Author

Choose a reason for hiding this comment

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

yeah for debugging CI, fails should exit with line number seems this one has moved a while without updating :D

@samwillis samwillis merged commit e09abec into main Aug 28, 2024
9 checks passed
@samwillis samwillis deleted the pmp-p/dbg-sockfiles branch August 28, 2024 18:26
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.

Abstract away console logging?
3 participants