Skip to content

lua*-luadbi: update to 0.7.5 - #63000

Open
Luciogi wants to merge 2 commits into
void-linux:masterfrom
Luciogi:luadbi-0.7
Open

Luciogi wants to merge 2 commits into
void-linux:masterfrom
Luciogi:luadbi-0.7

Conversation

@Luciogi

@Luciogi Luciogi commented Oct 13, 2026

Copy link
Copy Markdown
Contributor

Testing the changes

  • I tested the changes in this PR: YES

Local build testing

  • I built this PR locally for my native architecture, (X86_64-GLIBC)

@Luciogi
Luciogi force-pushed the luadbi-0.7 branch 6 times, most recently from 6458cb0 to 36a6500 Compare October 13, 2026 15:16

@Duncaen Duncaen left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

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

Why is the duckdb change in this PR and why is it needed?

@Duncaen

Duncaen commented Oct 14, 2026

Copy link
Copy Markdown
Member

Also don't really like calling make from pkg_install functions, the normal practice is to do this in steps, the intsall step is installing things into destdir and then the the package step splits sub packages.

@Luciogi

Luciogi commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor Author

Why is the duckdb change in this PR and why is it needed?

hook 04-generate-runtime-deps was failing as it could not found libduckdb.so in common/shlibs

previously libduckdb.so was packaged in duckdb-devel, but in shlibs file, i could not any instance where -devel package was added.

duckdb does not have .so file like other package haves libduckdb.so.4

@Luciogi

Luciogi commented Oct 15, 2026 •

Copy link
Copy Markdown
Contributor Author

Also don't really like calling make from pkg_install functions, the normal practice is to do this in steps, the intsall step is installing things into destdir and then the the package step splits sub packages.

If I recall correctly, after running `do_build()` it was producing object file in `/masterdir/buildir/lua54-luadbi-0.7.5/lua*/build/`

I may need to recheck.

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