/snapd

cmd: use libtool for the internal library #3162

Closed
wants to merge 1 commit into
from

Conversation

Reviewers

@mvo5 mvo5

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
3 participants
@morphis @mvo5 @zyga
@morphis
Contributor

morphis commented Apr 10, 2017

This just cuts some hassle integration with how the helper library is
built. I could never get to pass the right -fPIC/-fpie combination
otherwise.

Signed-off-by: Zygmunt Krynicki me@zygoon.pl
@mvo5

mvo5 approved these changes Apr 11, 2017
View changes

@mvo5 mvo5 added this to the 2.24 milestone Apr 11, 2017

@mvo5
Collaborator

mvo5 commented Apr 11, 2017

The test error in here look real though.
@zyga
cmd: use libtool for the internal library 
This just cuts some hassle integration with how the helper library is
built. I could never get to pass the right -fPIC/-fpie combination
otherwise.

Signed-off-by: Zygmunt Krynicki <me@zygoon.pl>

All checks have failed

6 failing checks
continuous-integration/travis-ci/pr — The Travis CI build failed
Details
@mvo5
xenial-amd64 — autopkgtest finished (failure)
Details
@mvo5
xenial-i386 — autopkgtest finished (failure)
Details
@mvo5
xenial-ppc64el — autopkgtest finished (failure)
Details
@mvo5
yakkety-amd64 — autopkgtest finished (failure)
Details
@mvo5
zesty-amd64 — autopkgtest finished (failure)
Details
687b6de
@morphis
Contributor

morphis commented Apr 11, 2017

@mvo5 in spread I only see

2017-04-11 05:40:06 Discarding linode:ubuntu-16.04-32 (Spread-06), cannot connect: cannot connect to linode:ubuntu-16.04-32 (Spread-06): ssh: handshake failed: ssh: unable to authenticate, attempted methods [none password], no supported methods remain
@morphis
Contributor

morphis commented Apr 11, 2017

Ok, the autopkgtest's highlight the problem:

/usr/lib/snapd/snap-confine: error while loading shared libraries: libcap.so.2: cannot open shared object file: No such file or directory

Looking into that now.
@morphis
Contributor

morphis commented Apr 11, 2017

Closing again as this is not going to work out as it should in a sane way for all platforms. Need to find a proper way how to deal with this problem.

@morphis morphis closed this Apr 11, 2017

@Conan-Kudo Conan-Kudo referenced this pull request May 21, 2017

Merged

packaging: Add Fedora packaging files #3366

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment