<pre style='margin:0'>
Herby Gillot (herbygillot) pushed a commit to branch master
in repository macports-ports.
</pre>
<p><a href="https://github.com/macports/macports-ports/commit/6cd05fba64554d14aadcc0e1ff954463ab762c56">https://github.com/macports/macports-ports/commit/6cd05fba64554d14aadcc0e1ff954463ab762c56</a></p>
<pre style="white-space: pre; background: #F8F8F8">The following commit(s) were added to refs/heads/master by this push:
<span style='display:block; white-space:pre;color:#404040;'> new 6cd05fba645 lima: fix file ownership
</span>6cd05fba645 is described below
<span style='display:block; white-space:pre;color:#808000;'>commit 6cd05fba64554d14aadcc0e1ff954463ab762c56
</span>Author: Dan Villiom Podlaski Christiansen <danchr@macports.org>
AuthorDate: Wed May 29 21:07:06 2024 +0200
<span style='display:block; white-space:pre;color:#404040;'> lima: fix file ownership
</span><span style='display:block; white-space:pre;color:#404040;'>
</span><span style='display:block; white-space:pre;color:#404040;'> This partially undoes an earlier applied patch to allow generating
</span><span style='display:block; white-space:pre;color:#404040;'> completions as root, performing it during the `build` step instead. In
</span><span style='display:block; white-space:pre;color:#404040;'> addition, the `install` target in the `Makefile` actually uses `tar` to
</span><span style='display:block; white-space:pre;color:#404040;'> copy the files, so we patch the Makefile to pass `--no-same-owner` to it
</span><span style='display:block; white-space:pre;color:#404040;'> so that it never retains ownership, even if run as root
</span>---
sysutils/lima/Portfile | 31 ++++++++++++++-----------------
sysutils/lima/files/patch-Makefile.diff | 9 +++++++++
sysutils/lima/files/patch-main.diff | 11 -----------
3 files changed, 23 insertions(+), 28 deletions(-)
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/sysutils/lima/Portfile b/sysutils/lima/Portfile
</span><span style='display:block; white-space:pre;color:#808080;'>index edd6735f2c7..2a4f4493632 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/sysutils/lima/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/sysutils/lima/Portfile
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -6,7 +6,7 @@ PortGroup golang 1.0
</span> go.setup github.com/lima-vm/lima 0.22.0 v
go.offline_build no
github.tarball_from archive
<span style='display:block; white-space:pre;background:#ffe0e0;'>-revision 0
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+revision 1
</span>
homepage https://lima-vm.io
<span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -36,31 +36,28 @@ checksums rmd160 a24c3a0a5623b2b759dec8dbe7ef5a0d2892af7b \
</span>
build.cmd make
<span style='display:block; white-space:pre;background:#ffe0e0;'>-patchfiles patch-Makefile.diff \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- patch-main.diff
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+patchfiles patch-Makefile.diff
</span>
post-patch {
reinplace "s|@@VERSION@@|${version}|g" ${worksrcpath}/Makefile
}
<span style='display:block; white-space:pre;background:#ffe0e0;'>-destroot {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- system -W ${worksrcpath} "make install DEST=${destroot}${prefix}"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+post-build {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set limactl ${worksrcpath}/_output/bin/limactl
</span>
<span style='display:block; white-space:pre;background:#ffe0e0;'>- # install completion for various shells
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- set bash_completions_dir ${destroot}${prefix}/share/bash-completion/completions
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- set fish_completions_dir ${destroot}${prefix}/share/fish/vendor_completions.d
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- set zsh_completions_dir ${destroot}${prefix}/share/zsh/site-functions
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set bash_completions_dir ${worksrcpath}/_output/share/bash-completion/completions
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set fish_completions_dir ${worksrcpath}/_output/share/fish/vendor_completions.d
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ set zsh_completions_dir ${worksrcpath}/_output/share/zsh/site-functions
</span>
xinstall -d ${bash_completions_dir} ${fish_completions_dir} ${zsh_completions_dir}
<span style='display:block; white-space:pre;background:#ffe0e0;'>- system "
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- ${destroot}${prefix}/bin/limactl completion bash \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- > ${bash_completions_dir}/${name}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- ${destroot}${prefix}/bin/limactl completion fish \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- > ${fish_completions_dir}/${name}.fish
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- ${destroot}${prefix}/bin/limactl completion zsh \
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- > ${zsh_completions_dir}/_${name}
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- "
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ system "${limactl} completion bash > ${bash_completions_dir}/${name}"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ system "${limactl} completion fish > ${fish_completions_dir}/${name}.fish"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ system "${limactl} completion zsh > ${zsh_completions_dir}/_${name}"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+}
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+destroot {
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ system -W ${worksrcpath} "make install DEST=${destroot}${prefix}"
</span> }
github.livecheck.regex {([0-9.]+)}
<span style='display:block; white-space:pre;color:#808080;'>diff --git a/sysutils/lima/files/patch-Makefile.diff b/sysutils/lima/files/patch-Makefile.diff
</span><span style='display:block; white-space:pre;color:#808080;'>index 15f8c1e5a99..d70413771b2 100644
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/sysutils/lima/files/patch-Makefile.diff
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ b/sysutils/lima/files/patch-Makefile.diff
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -9,3 +9,12 @@
</span> VERSION_TRIMMED := $(VERSION:v%=%)
GO_BUILD := $(GO) build -ldflags="-s -w -X $(PACKAGE)/pkg/version.Version=$(VERSION)" -tags "$(GO_BUILDTAGS)"
<span style='display:block; white-space:pre;background:#e0ffe0;'>+@@ -209,7 +209,7 @@
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ install: uninstall
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ mkdir -p "$(DEST)"
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ # Use tar rather than cp, for better symlink handling
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+- ( cd _output && tar c * | tar Cxv "$(DEST)" )
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>++ ( cd _output && $(TAR) c * | $(TAR) -xv --no-same-owner -C "$(DEST)" )
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if [ "$(shell uname -s )" != "Linux" -a ! -e "$(DEST)/bin/nerdctl" ]; then ln -sf nerdctl.lima "$(DEST)/bin/nerdctl"; fi
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+ if [ "$(shell uname -s )" != "Linux" -a ! -e "$(DEST)/bin/apptainer" ]; then ln -sf apptainer.lima "$(DEST)/bin/apptainer"; fi
</span><span style='display:block; white-space:pre;background:#e0ffe0;'>+
</span><span style='display:block; white-space:pre;color:#808080;'>diff --git a/sysutils/lima/files/patch-main.diff b/sysutils/lima/files/patch-main.diff
</span>deleted file mode 100644
<span style='display:block; white-space:pre;color:#808080;'>index 84b726caa2e..00000000000
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>--- a/sysutils/lima/files/patch-main.diff
</span><span style='display:block; white-space:pre;background:#e0e0ff;'>+++ /dev/null
</span><span style='display:block; white-space:pre;background:#e0e0e0;'>@@ -1,11 +0,0 @@
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>---- cmd/limactl/main.go
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-+++ cmd/limactl/main.go
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-@@ -87,7 +87,7 @@ func newApp() *cobra.Command {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- formatter.ForceColors = true
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- logrus.StandardLogger().SetFormatter(formatter)
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-- if os.Geteuid() == 0 && cmd.Name() != "generate-doc" {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>-+ if os.Geteuid() == 0 && cmd.Name() != "generate-doc" && cmd.Parent().Name() != "completion" {
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- return errors.New("must not run as the root")
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- }
</span><span style='display:block; white-space:pre;background:#ffe0e0;'>- // Make sure either $HOME or $LIMA_HOME is defined, so we don't need
</span></pre><pre style='margin:0'>
</pre>