Yes, I'll take the ownership for now. Thanks.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2414#issuecomment-2017947916
You are receiving this because you are subscribed to this thread.
Message ID:
This is entirely non-obvious as one would assume that any exported env vars are
inherited by the runroot() subprocesses. Yet they arent because we
intentionally reset the environment in the bwrap containers with --clearenv
(see atlocal.in). Its a common use case, though, so deserves a proper
Merged #2991 into master.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/2991#event-12232880576
You are receiving this because you are subscribed to this thread.
Message ID:
___
Rpm-maint
@pmatilai commented on this pull request.
> +if rpm.expand('%[0%{?_use_weak_usergroup_deps}]') ~= '0' then
+ return
+end
+for line in io.lines(macros["1"]) do
+if line:sub(1, 1) == '#' then
+goto continue
+end
+fields = {}
+for w in
@pmatilai commented on this pull request.
> +if rpm.expand('%[0%{?_use_weak_usergroup_deps}]') ~= '0' then
+ return
+end
+for line in io.lines(macros["1"]) do
+if line:sub(1, 1) == '#' then
+goto continue
+end
+fields = {}
+for w in
@pmatilai commented on this pull request.
> @@ -20,3 +20,41 @@
::continue::
end
}
+
+%__sysusers_recommends() %{lua:
+if rpm.expand('%[0%{?_use_weak_usergroup_deps}]') == '0' then
Oh and there's indeed a nicer way to do this (I remember you asking about this
but didn't have
I'll open a pull request for this.
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2972#issuecomment-2017876840
You are receiving this because you are subscribed to this thread.
Message ID:
Look for per-user macros primarily in ${XDG_CONFIG_HOME}/rpmmacros but fall
back to traditional ~/.rpmmacros iff it exists and theres no config in the
XDG location. As per the XDG spec, if ${XDG_CONFIG_HOME} is not set, it
defaults to ~/.config
Fixes: #2153
You can view, comment on, or merge
@pmatilai commented on this pull request.
> +fields = {}
+for w in line:gmatch("%S+") do
+table.insert(fields, w)
+end
+if #fields >= 3 and fields[1] == 'm' then
+ print(string.format('user(%s)\\ngroup(%s)', fields[2], fields[3]))
+
Add support for sysusers group membership lines
m user group
Create Requires/Recommends for both the user and the group.
You can view, comment on, or merge this pull request online at:
https://github.com/rpm-software-management/rpm/pull/2990
-- Commit Summary --
* Re-Word User / Group
These two commits, ready for inclusion, document that the `--macros` option
accepts globs as well and fix a warning caused by groff not being able to
adjust (justify) the line(s) with the default value for the `--macros` option.
You can view, comment on, or merge this pull request online at:
Our headers are always useing utf8 and the pax standard also requires utf8
strings. So do this nasty little locale switching to make libarchive not depend
on the active locale.
You can view, comment on, or merge this pull request online at:
@Conan-Kudo requested changes on this pull request.
> +#ifdef MACROFILES
+static char *initMacroPath(const char *confdir)
+{
+return xstrdup(MACROFILES);
+}
+#else
+/*
+ * Prefer XDG_CONFIG_HOME/rpmmacros but fall back to ~/.rpmmacros
+ * if it exists and the XDG path doesn't.
+ */
+static
@dmnks commented on this pull request.
> +#ifdef MACROFILES
+static char *initMacroPath(const char *confdir)
+{
+return xstrdup(MACROFILES);
+}
+#else
+/*
+ * Prefer XDG_CONFIG_HOME/rpmmacros but fall back to ~/.rpmmacros
+ * if it exists and the XDG path doesn't.
+ */
+static char
Found out about this new feature. How does one go about constructing such
`BuildSystem`, do they just define a `%buildsystem__conf` macros?
I am considering the case of [MPI
packaging](https://pagure.io/packaging-committee/issue/1345) and it would be a
neat approach to `for` looping all mpi
@pmatilai pushed 3 commits.
ee17de74cf23df317e9c53f259c9bdd483a2ff4d Use rpmGlobPath(... RPMGLOB_NOCHECK)
for rpmrc reading
3b796dc90e819fe7cfaa621cd7e8d4f62c6c517d Sanitize rpmGlob() behavior wrt
non-glob patterns
1172a0f60e336891a86df77de89e3e82d536b39f Support per-user macro configuration
Okay, made you the admin of that repo. Have fun, as they say :sweat_smile:
--
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/issues/2414#issuecomment-2018081307
You are receiving this because you are subscribed to this thread.
Message ID:
@dmnks requested changes on this pull request.
> @@ -2,6 +2,41 @@
#
AT_BANNER([RPM macros])
+AT_SETUP([macro path])
+AT_KEYWORDS([macros])
+RPMDB_INIT
+
+# .rpmmacros exists
I'd suggest that we actually check that `~/.rpmmacros` really exists (or just
`touch` it like in the other tests
18 matches
Mail list logo