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

Build appstreamcli static #5

Closed
wants to merge 13 commits into from
Closed

Build appstreamcli static #5

wants to merge 13 commits into from

Conversation

probonopd
Copy link
Owner

@probonopd probonopd commented Aug 15, 2020

@probonopd
Copy link
Owner Author

Getting

/usr/lib/gcc/x86_64-alpine-linux-musl/8.3.0/../../../../x86_64-alpine-linux-musl/bin/ld: attempted static link of dynamic object `src/libappstream.so.0.12.9'

@probonopd
Copy link
Owner Author

Now getting

attempted static link of dynamic object `/usr/lib/libglib-2.0.so'

@probonopd
Copy link
Owner Author

Now getting

meson.build:63:0: ERROR: Unknown variable "static".

@probonopd
Copy link
Owner Author

meson.build:63:0: ERROR: All keyword arguments must be after positional arguments.

This means that instead of things like

glib_dep = dependency(static: true, 'glib-2.0', version: '>=2.58')

we probably need

glib_dep = dependency('glib-2.0', version: '>=2.58', static: true)

@probonopd
Copy link
Owner Author

Getting

meson.build:63:11: ERROR: Expecting rparen got comma.
('glib-2.0', version : '>=2.54', static: true)
^__________^

Why?

@probonopd probonopd added the help wanted Extra attention is needed label Aug 15, 2020
@Tachi107 Tachi107 mentioned this pull request Apr 29, 2022
@probonopd probonopd closed this in 6662b61 Apr 30, 2022
@Tachi107
Copy link
Contributor

You might want to delete this branch

@probonopd probonopd deleted the static-appstreamcli branch April 30, 2022 13:58
@probonopd
Copy link
Owner Author

Indeed. Thanks @Tachi107

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
help wanted Extra attention is needed
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants