Skip to content

Update download script#14

Merged
adsharma merged 2 commits into
masterfrom
update_download_script
Apr 5, 2026
Merged

Update download script#14
adsharma merged 2 commits into
masterfrom
update_download_script

Conversation

@adsharma
Copy link
Copy Markdown
Contributor

@adsharma adsharma commented Apr 4, 2026

Fixes: #13

@adsharma adsharma force-pushed the update_download_script branch from d6ecd8a to 5820ded Compare April 4, 2026 16:02
Comment thread example/main.go Outdated
@emehrkay
Copy link
Copy Markdown

emehrkay commented Apr 4, 2026

Still doesn't work as described in the README

  • I pull this branch
  • Created a new project with a go mod init
  • Copied the go example from https://docs.ladybugdb.com/get-started/
  • Added the go generate command at the top of the file //go:generate sh -c "cd $(go list -f '{{.Dir}}' -m github.com/LadybugDB/go-ladybug) && go generate ./..."
  • From within my project, ran go work init . ~/src/go/go-ladybug/ which created a go.work file that points to my local go-ladybug
  • Ran go generate ./...
  • It created a lib folder in the same directory as my go-ladybug
  • That lib folder has lbug.h lbug.hpp and liblbug.dylib
  • Run go build
  • It complains that ld: library 'lbug' not found in /go-ladybug/lib/dynamic/osx`
  • Move the contents of the lib directory to /go-ladybug/lib/dynamic/osx
  • Run go build
  • execute the executable
  • Error go-ladybug/lib/dynamic/osx/liblbug.0.dylib
  • Rename liblbug.dylib to liblbug.0.dylib
  • Rerun executable
  • Things work as expected

@adsharma adsharma force-pushed the update_download_script branch 3 times, most recently from 3609ae5 to 91a489a Compare April 4, 2026 23:56
@adsharma adsharma force-pushed the update_download_script branch from 91a489a to 03c5587 Compare April 4, 2026 23:57
@adsharma
Copy link
Copy Markdown
Contributor Author

adsharma commented Apr 5, 2026

download_lbug.sh now calls the upstream script to fetch the archives. CI is passing. Please review the updated README.

@emehrkay
Copy link
Copy Markdown

emehrkay commented Apr 5, 2026

Both ways work! Thanks

@adsharma adsharma merged commit 3ea5d54 into master Apr 5, 2026
4 checks passed
@adsharma adsharma deleted the update_download_script branch April 5, 2026 16:37
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.

macOS: Setup fails because the download_lbug.sh script wants to download liblbug-osx-universal.tar.gz which doesn't exist

2 participants