All of lore.kernel.org
 help / color / mirror / Atom feed
From: Mark Asselstine <mark.asselstine@windriver.com>
To: Bruce Ashfield <bruce.ashfield@gmail.com>
Cc: meta-virtualization <meta-virtualization@yoctoproject.org>,
	Stefan Agner <stefan.agner@toradex.com>
Subject: Re: [PATCH 4/6] skopeo: add skopeo
Date: Tue, 3 Sep 2019 17:05:59 -0400	[thread overview]
Message-ID: <1771246.8WWc8Gzl8P@yow-masselst-lx1> (raw)
In-Reply-To: <CADkTA4PBvoEP7w-Q1QDCDjE6+dX6SzxFF-_jrbYZ+A11gL7kFw@mail.gmail.com>

On Tuesday, September 3, 2019 5:00:18 P.M. EDT Bruce Ashfield wrote:
> On Tue, Sep 3, 2019 at 4:41 PM Mark Asselstine
> 
> <mark.asselstine@windriver.com> wrote:
> > On Tue, Sep 3, 2019 at 1:18 PM Bruce Ashfield <bruce.ashfield@gmail.com> 
wrote:
> > > On Sun, Sep 1, 2019 at 4:35 PM Stefan Agner <stefan@agner.ch> wrote:
> > > > From: Stefan Agner <stefan.agner@toradex.com>
> > > > 
> > > > skopeo is a command line utility that performs various operations on
> > > > container images and image repositories.
> > > > 
> > > > skopeo can work with OCI images as well as the original Docker v2
> > > > images.
> > > > 
> > > > The recipe originates from from meta-overc commit a497792. It has
> > > > been updated with the new project URL and v0.1.39.
> > > 
> > > Nice!
> > > 
> > > This was on my list to bring into meta-virtualization. Once I can
> > > confirm that it passes the meta-overc use cases, I'll merge this and
> > > drop the one from meta-overc.
> > 
> > Which takes an item off my todo list. So double nice!
> > 
> > Eventually I want to have this build -native and an addition to the
> > fetcher so it can be used to populate a rootfs with container images
> > from something like docker hub.
> 
> I can honestly say "good luck wit that". There has to be an easier
> route than skopeo for that functionality.
> 
> It simply has too many dependencies (and yet unused functionality) to
> be built -native. IMHO, it's just not the right tool for that job.
> 

For sure and knowing that you have spent quite a bit of time looking at this I 
am not about to argue against your assessment. I don't think this closes the 
door on working with the skopeo folks to break up the functionality in order 
to make this more feasible. At any rate, this is a ways off.

MarkA

> Bruce
> 
> > MarkA
> > 
> > > Bruce
> > > 
> > > > Signed-off-by: Stefan Agner <stefan.agner@toradex.com>
> > > > ---
> > > > 
> > > >  recipes-containers/skopeo/skopeo_git.bb | 73
> > > >  +++++++++++++++++++++++++
> > > >  1 file changed, 73 insertions(+)
> > > >  create mode 100644 recipes-containers/skopeo/skopeo_git.bb
> > > > 
> > > > diff --git a/recipes-containers/skopeo/skopeo_git.bb
> > > > b/recipes-containers/skopeo/skopeo_git.bb new file mode 100644
> > > > index 0000000..82d3f23
> > > > --- /dev/null
> > > > +++ b/recipes-containers/skopeo/skopeo_git.bb
> > > > @@ -0,0 +1,73 @@
> > > > +HOMEPAGE = "https://github.com/containers/skopeo"
> > > > +SUMMARY = "Work with remote images registries - retrieving
> > > > information, images, signing content" +LICENSE = "Apache-2.0"
> > > > +LIC_FILES_CHKSUM =
> > > > "file://src/import/LICENSE;md5=7e611105d3e369954840a6668c438584" +
> > > > +DEPENDS = " \
> > > > +    gpgme \
> > > > +    multipath-tools \
> > > > +    btrfs-tools \
> > > > +    glib-2.0 \
> > > > +    ostree \
> > > > +"
> > > > +
> > > > +inherit go
> > > > +
> > > > +RDEPENDS_${PN} = " \
> > > > +     gpgme \
> > > > +     libgpg-error \
> > > > +     libassuan \
> > > > +"
> > > > +
> > > > +SRC_URI = "git://github.com/containers/skopeo"
> > > > +
> > > > +SRCREV = "1cf1e06582142c522543560f2bc6d6756696e8ad"
> > > > +PV = "v0.1.39-dev+git${SRCPV}"
> > > > +GO_IMPORT = "import"
> > > > +
> > > > +S = "${WORKDIR}/git"
> > > > +
> > > > +inherit goarch
> > > > +inherit pkgconfig
> > > > +
> > > > +# This disables seccomp and apparmor, which are on by default in the
> > > > +# go package.
> > > > +EXTRA_OEMAKE="BUILDTAGS=''"
> > > > +
> > > > +do_compile() {
> > > > +       export GOARCH="${TARGET_GOARCH}"
> > > > +
> > > > +       # Setup vendor directory so that it can be used in GOPATH.
> > > > +       #
> > > > +       # Go looks in a src directory under any directory in GOPATH
> > > > but riddler +       # uses 'vendor' instead of 'vendor/src'. We can
> > > > fix this with a symlink. +       #
> > > > +       # We also need to link in the ipallocator directory as that is
> > > > not under +       # a src directory.
> > > > +       ln -sfn . "${S}/src/import/vendor/src"
> > > > +       mkdir -p
> > > > "${S}/src/import/vendor/src/github.com/projectatomic/skopeo" +      
> > > > ln -sfn "${S}/src/import/skopeo"
> > > > "${S}/src/import/vendor/src/github.com/projectatomic/skopeo" +      
> > > > ln -sfn "${S}/src/import/version"
> > > > "${S}/src/import/vendor/src/github.com/projectatomic/skopeo/version"
> > > > +       export GOPATH="${S}/src/import/vendor"
> > > > +
> > > > +       # Pass the needed cflags/ldflags so that cgo
> > > > +       # can find the needed headers files and libraries
> > > > +       export CGO_ENABLED="1"
> > > > +       export CFLAGS=""
> > > > +       export LDFLAGS=""
> > > > +       export CGO_CFLAGS="${BUILDSDK_CFLAGS}
> > > > --sysroot=${STAGING_DIR_TARGET}" +       export
> > > > CGO_LDFLAGS="${BUILDSDK_LDFLAGS} --sysroot=${STAGING_DIR_TARGET}" +  
> > > >     cd ${S}/src/import
> > > > +
> > > > +       oe_runmake binary-local
> > > > +}
> > > > +
> > > > +do_install() {
> > > > +       install -d ${D}/${sbindir}
> > > > +       install -d ${D}/${sysconfdir}/containers
> > > > +
> > > > +       install ${S}/src/import/skopeo ${D}/${sbindir}/
> > > > +       install ${S}/src/import/default-policy.json
> > > > ${D}/${sysconfdir}/containers/policy.json +}
> > > > +
> > > > +INSANE_SKIP_${PN} += "ldflags"
> > > > --
> > > > 2.20.1
> > > > 
> > > > --
> > > > _______________________________________________
> > > > meta-virtualization mailing list
> > > > meta-virtualization@yoctoproject.org
> > > > https://lists.yoctoproject.org/listinfo/meta-virtualization
> > > 
> > > --
> > > - Thou shalt not follow the NULL pointer, for chaos and madness await
> > > thee at its end
> > > - "Use the force Harry" - Gandalf, Star Trek II
> > > --
> > > _______________________________________________
> > > meta-virtualization mailing list
> > > meta-virtualization@yoctoproject.org
> > > https://lists.yoctoproject.org/listinfo/meta-virtualization






  reply	other threads:[~2019-09-03 21:06 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-09-01 20:33 [PATCH 0/6] podman: add initial recipe Stefan Agner
2019-09-01 20:33 ` [PATCH 1/6] conmon: initial add Stefan Agner
2019-09-01 20:33 ` [PATCH 2/6] go-metalinter: add recipe for gometalinter Stefan Agner
2019-09-01 20:33 ` [PATCH 3/6] go-md2man: add md2man Stefan Agner
2019-09-01 20:33 ` [PATCH 4/6] skopeo: add skopeo Stefan Agner
2019-09-03 17:18   ` Bruce Ashfield
2019-09-03 20:41     ` Mark Asselstine
2019-09-03 21:00       ` Bruce Ashfield
2019-09-03 21:05         ` Mark Asselstine [this message]
2019-09-03 21:13           ` Bruce Ashfield
2019-09-03 21:29             ` Mark Asselstine
2019-09-04  6:55             ` Stefan Agner
2019-09-04 12:40               ` Bruce Ashfield
2019-09-01 20:33 ` [PATCH 5/6] cni: move cni tools to /usr/lib/cni Stefan Agner
2019-09-02  1:45   ` ChenQi
2019-09-03 17:28     ` Bruce Ashfield
2019-09-06 12:06       ` Stefan Agner
2019-09-06 12:42         ` Bruce Ashfield
2019-09-09 21:36           ` Stefan Agner
2019-09-01 20:33 ` [PATCH 6/6] podmon: add the Pod Manager tool Stefan Agner
2019-09-03 17:26   ` Bruce Ashfield
2019-09-09 21:30     ` Stefan Agner
2019-09-10 23:13       ` akuster
2019-09-03 17:20 ` [PATCH 0/6] podman: add initial recipe Bruce Ashfield
2019-09-04  6:44   ` Stefan Agner
2019-09-04 12:35     ` Bruce Ashfield

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1771246.8WWc8Gzl8P@yow-masselst-lx1 \
    --to=mark.asselstine@windriver.com \
    --cc=bruce.ashfield@gmail.com \
    --cc=meta-virtualization@yoctoproject.org \
    --cc=stefan.agner@toradex.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.