* Apply patch to read-only file
@ 2021-09-16 7:49 phodina via
2021-09-25 12:08 ` Guillaume Le Vaillant
0 siblings, 1 reply; 6+ messages in thread
From: phodina via @ 2021-09-16 7:49 UTC (permalink / raw)
To: help-guix
Hi,
I'm attempting to package realmd program. However, there seems to be an issue as it does not support GuixSD. Therefore I created a patch.
When I specify the patch the hunk fails as the configure.ac file is read-only.
I tried to use snippets but they seem to be applied after the patch therefore don't work.
Is there a way how to patch a read-only files? I got the idea from this thread [1]. Below is the package definition and patch.
[1] https://lists.gnu.org/archive/html/help-guix/2020-01/msg00063.html
realmd.scm
-------------------------------------------------------------------------------------
(define-module (realmd)
#:use-module (guix packages)
#:use-module (gnu packages)
#:use-module (gnu packages image)
#:use-module (gnu packages glib)
#:use-module (gnu packages autotools)
#:use-module (gnu packages pkg-config)
#:use-module (gnu packages polkit)
#:use-module (guix git-download)
#:use-module (guix build-system gnu)
#:use-module ((guix licenses) #:prefix license:))
(define-public realmd
(package
(name "realmd")
(version "0.17.0")
(source (origin
(method git-fetch)
(uri (git-reference
(url "https://github.com/freedesktop/realmd")
(commit version)))
(file-name (git-file-name name version))
(modules '((guix build utils)))
(snippet
'(begin
(delete-file "configure.ac")
;(make-file-writable "configure.ac")
#t))
(patches '("realmd-guixsd-support.patch"))
(sha256
(base32
"1c6q2a86kk2f1akzc36nh52hfwsmmc0mbp6ayyjxj4zsyk9zx5bf"))))
(build-system gnu-build-system)
(native-inputs `(("pkg-config" ,pkg-config)
("autoconf" ,autoconf)
("intltool" ,intltool)
("automake" ,automake)))
(inputs `(("glib" ,glib)
("polkit" ,polkit)))
(synopsis "DBus service for configuring kerberos and other online identities")
(description "Dbus system service that manages discovery and enrollment
in realms/domains like Active Directory or IPA.")
(home-page "https://www.freedesktop.org/software/realmd/")
(license license:lgpl2.1)))
------------------------------------------------------------------------------
realmd-guixsd-support.patch ([PATCH] Add GuixSD support)
-------------------------------------------------------------------------------
diff --git a/configure.ac b/configure.ac
index 4dac5a9..9e818d6 100644
--- a/configure.ac
+++ b/configure.ac
@@ -38,6 +38,7 @@ if test -z $DISTRO; then
AC_CHECK_FILE(/etc/redhat-release, [DISTRO="redhat"])
AC_CHECK_FILE(/etc/debian_version, [DISTRO="debian"])
AC_CHECK_FILE(/etc/SuSE-release, [DISTRO="suse"])
+ AC_CHECK_FILE(/gnu/store, [DISTRO="guixsd"])
# Not customized for these yet
dnl AC_CHECK_FILE(/etc/gentoo-release, [DISTRO="gentoo"])
@@ -103,51 +104,6 @@ PKG_CHECK_MODULES(POLKIT, polkit-gobject-1)
AC_SUBST(POLKIT_CFLAGS)
AC_SUBST(POLKIT_LIBS)
-# --------------------------------------------------------------------
-# systemd
-
-AC_MSG_CHECKING([systemd unit directory])
-AC_ARG_WITH(systemd-unit-dir,
- AS_HELP_STRING([--with-systemd-unit-dir],
- [Directory to install systemd service file]))
-
-if test "$with_systemd_unit_dir" = "" -o "$with_systemd_unit_dir" = "yes"; then
- with_systemd_unit_dir=$($PKG_CONFIG --variable=systemdsystemunitdir systemd)
- if test "$with_systemd_unit_dir" = ""; then
- AC_MSG_ERROR(["Couldn't find systemd system unit directory"])
- fi
-
-elif test "$with_systemd_unit_dir" = "no"; then
- with_systemd_unit_dir=""
-
-fi
-
-SYSTEMD_UNIT_DIR="$with_systemd_unit_dir"
-AC_SUBST(SYSTEMD_UNIT_DIR)
-AM_CONDITIONAL(WITH_SYSTEMD, [test -n "$with_systemd_unit_dir"])
-
-dbus_systemd_service=""
-if test -n "$with_systemd_unit_dir"; then
- dbus_systemd_service="SystemdService=realmd.service"
-fi
-AC_SUBST(dbus_systemd_service)
-
-AC_MSG_RESULT($with_systemd_unit_dir)
-
-AC_ARG_WITH(systemd-journal,
- AS_HELP_STRING([--with-systemd-journal],
- [Use systemd's journal for logging]))
-
-if test "$with_systemd_journal" != "no"; then
- AC_DEFINE_UNQUOTED(WITH_JOURNAL, 1, [Use systemd's journal])
- PKG_CHECK_MODULES(SYSTEMD_JOURNAL, [libsystemd],,
- [PKG_CHECK_MODULES(SYSTEMD_JOURNAL, [libsystemd-journal])])
- with_systemd_journal="yes"
-fi
-
-AC_SUBST(SYSTEMD_JOURNAL_CFLAGS)
-AC_SUBST(SYSTEMD_JOURNAL_LIBS)
-
# -------------------------------------------------------------------
# resolv
diff --git a/service/realmd-guixsd.conf b/service/realmd-guixsd.conf
new file mode 100644
index 0000000..ac4b436
--- /dev/null
+++ b/service/realmd-guixsd.conf
@@ -0,0 +1,42 @@
+# Default values for realmd
+[service]
+debug = no
+automatic-install = yes
+
+[paths]
+net = /usr/bin/net
+winbindd = /usr/sbin/winbindd
+smb.conf = /etc/smb.conf
+sssd.conf = /etc/sssd/sssd.conf
+adcli = /usr/sbin/adcli
+ipa-client-install = /usr/sbin/ipa-client-install
+pam_winbind.conf = /etc/security/pam_winbind.conf
+krb5.conf = /etc/krb5.conf
+
+[active-directory]
+default-client = sssd
+os-name =
+os-version =
+
+[providers]
+sssd = yes
+samba = yes
+example = no
+
+[samba-packages]
+
+[winbind-packages]
+
+[sssd-packages]
+
+[adcli-packages]
+
+[commands]
+
+[users]
+default-shell = /bin/bash
+default-home = /home/%U@%D
+
+[example.com]
+example-administrator = Administrator
+example-password = bureaucracy
--
2.32.0
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: Apply patch to read-only file
2021-09-16 7:49 Apply patch to read-only file phodina via
@ 2021-09-25 12:08 ` Guillaume Le Vaillant
2021-09-27 22:57 ` phodina
0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Le Vaillant @ 2021-09-25 12:08 UTC (permalink / raw)
To: phodina; +Cc: help-guix
[-- Attachment #1: Type: text/plain, Size: 287 bytes --]
Hi,
It looks like your email client removed many spaces in your definitions
for "realmd.scm" and "realmd-guixsd-support.patch", which made the
package definition hard to read and the patch invalid.
Could send these files as attachment (or prevent your email client from
mangling them)?
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Apply patch to read-only file
2021-09-25 12:08 ` Guillaume Le Vaillant
@ 2021-09-27 22:57 ` phodina
2021-09-29 8:42 ` Guillaume Le Vaillant
0 siblings, 1 reply; 6+ messages in thread
From: phodina @ 2021-09-27 22:57 UTC (permalink / raw)
To: Guillaume Le Vaillant; +Cc: help-guix
Hi Guillaume,
Sure, here's the formatted file. My appology for the issue with the mail client.
--8<---------------cut here---------------start------------->8---
(define-module (realmd)
#:use-module (guix packages)
#:use-module (gnu packages)
#:use-module (gnu packages image)
#:use-module (gnu packages glib)
#:use-module (gnu packages autotools)
#:use-module (gnu packages pkg-config)
#:use-module (gnu packages polkit)
#:use-module (guix git-download)
#:use-module (guix build-system gnu)
#:use-module ((guix licenses) #:prefix license:))
(define-public realmd
(package
(name "realmd")
(version "0.17.0")
(source (origin
(method git-fetch)
(uri (git-reference
(url "https://github.com/freedesktop/realmd")
(commit version)))
(file-name (git-file-name name version))
(modules '((guix build utils)))
(snippet
'(begin
(delete-file "configure.ac")
;(make-file-writable "configure.ac")
#t))
(patches '("endgame/packages/patches/realmd-guixsd-support.patch"))
(sha256
(base32
"1c6q2a86kk2f1akzc36nh52hfwsmmc0mbp6ayyjxj4zsyk9zx5bf"))))
(build-system gnu-build-system)
(arguments
`(#:phases
(modify-phases %standard-phases
(add-after 'unpack 'make-git-checkout-writable
(lambda* (#:key inputs #:allow-other-keys)
(make-file-writable "configure.ac"))
(add-after 'unpack 'apply-patches
(for-each
(lambda (file)
(invoke "patch" "--force" "-p1" "-input" "endgame/packages/patches/realmd-guixsd-support.patch"))))
))))
; gnuzilla.scm
; jami.scm
(native-inputs `(("pkg-config" ,pkg-config)
("autoconf" ,autoconf)
("intltool" ,intltool)
("automake" ,automake)))
(inputs `(("glib" ,glib)
("polkit" ,polkit)))
(synopsis "DBus service for configuring kerberos and other online identities")
(description "Dbus system service that manages discovery and enrollment
in realms/domains like Active Directory or IPA.")
(home-page "https://www.freedesktop.org/software/realmd/")
(license license:lgpl2.1)))
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Apply patch to read-only file
2021-09-27 22:57 ` phodina
@ 2021-09-29 8:42 ` Guillaume Le Vaillant
2021-10-02 19:49 ` phodina
0 siblings, 1 reply; 6+ messages in thread
From: Guillaume Le Vaillant @ 2021-09-29 8:42 UTC (permalink / raw)
To: phodina; +Cc: Sarah Morgensen, help-guix
[-- Attachment #1.1: Type: text/plain, Size: 203 bytes --]
Hi,
I tried with simplified versions of your package definition and patch,
and it builds fine for me. The patch gets applied successfully.
However I haven't tried using the program to see if it works.
[-- Attachment #1.2: realmd.scm --]
[-- Type: application/octet-stream, Size: 1890 bytes --]
(define-module (gnu packages realmd)
#:use-module (guix packages)
#:use-module (gnu packages)
#:use-module (gnu packages autotools)
#:use-module (gnu packages glib)
#:use-module (gnu packages image)
#:use-module (gnu packages kerberos)
#:use-module (gnu packages openldap)
#:use-module (gnu packages pkg-config)
#:use-module (gnu packages polkit)
#:use-module (gnu packages python)
#:use-module (guix git-download)
#:use-module (guix build-system gnu)
#:use-module ((guix licenses) #:prefix license:))
(define-public realmd
(package
(name "realmd")
(version "0.17.0")
(source (origin
(method git-fetch)
(uri (git-reference
(url "https://github.com/freedesktop/realmd")
(commit version)))
(file-name (git-file-name name version))
(sha256
(base32
"1c6q2a86kk2f1akzc36nh52hfwsmmc0mbp6ayyjxj4zsyk9zx5bf"))
(patches (search-patches "realmd-guixsd-support.patch"))))
(build-system gnu-build-system)
(native-inputs
`(("autoconf" ,autoconf)
("automake" ,automake)
("glib-bin" ,glib "bin")
("intltool" ,intltool)
("pkg-config" ,pkg-config)
("python" ,python)))
(inputs
`(("glib" ,glib)
("mit-krb5" ,mit-krb5)
("openldap" ,openldap)
("polkit" ,polkit)))
(arguments
`(#:configure-flags '("--with-systemd-unit-dir=no"
"--with-systemd-journal=no"
"--disable-doc")))
(synopsis
"DBus service for configuring kerberos and other online identities")
(description
"Dbus system service that manages discovery and enrollment in
realms/domains like Active Directory or IPA.")
(home-page "https://www.freedesktop.org/software/realmd/")
(license license:lgpl2.1)))
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #1.3: realmd-guixsd-support.patch --]
[-- Type: text/x-patch, Size: 1589 bytes --]
commit ffa79f48ce454b00c0590432010dbe58cdfd3ed2
Author: Guillaume Le Vaillant <glv@posteo.net>
Date: Wed Sep 29 10:17:50 2021 +0200
Add GuixSD support
diff --git a/configure.ac b/configure.ac
index ee067d9..27dea11 100644
--- a/configure.ac
+++ b/configure.ac
@@ -38,7 +38,8 @@ if test -z $DISTRO; then
AC_CHECK_FILE(/etc/redhat-release, [DISTRO="redhat"])
AC_CHECK_FILE(/etc/debian_version, [DISTRO="debian"])
AC_CHECK_FILE(/etc/SuSE-release, [DISTRO="suse"])
+ AC_CHECK_FILE(/gnu/store, [DISTRO="guixsd"])
# Not customized for these yet
dnl AC_CHECK_FILE(/etc/gentoo-release, [DISTRO="gentoo"])
diff --git a/service/realmd-guixsd.conf b/service/realmd-guixsd.conf
new file mode 100644
index 0000000..ac4b436
--- /dev/null
+++ b/service/realmd-guixsd.conf
@@ -0,0 +1,42 @@
+# Default values for realmd
+[service]
+debug = no
+automatic-install = yes
+
+[paths]
+net = /usr/bin/net
+winbindd = /usr/sbin/winbindd
+smb.conf = /etc/smb.conf
+sssd.conf = /etc/sssd/sssd.conf
+adcli = /usr/sbin/adcli
+ipa-client-install = /usr/sbin/ipa-client-install
+pam_winbind.conf = /etc/security/pam_winbind.conf
+krb5.conf = /etc/krb5.conf
+
+[active-directory]
+default-client = sssd
+os-name =
+os-version =
+
+[providers]
+sssd = yes
+samba = yes
+example = no
+
+[samba-packages]
+
+[winbind-packages]
+
+[sssd-packages]
+
+[adcli-packages]
+
+[commands]
+
+[users]
+default-shell = /bin/bash
+default-home = /home/%U@%D
+
+[example.com]
+example-administrator = Administrator
+example-password = bureaucracy
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 247 bytes --]
^ permalink raw reply related [flat|nested] 6+ messages in thread
* Re: Apply patch to read-only file
2021-09-29 8:42 ` Guillaume Le Vaillant
@ 2021-10-02 19:49 ` phodina
0 siblings, 0 replies; 6+ messages in thread
From: phodina @ 2021-10-02 19:49 UTC (permalink / raw)
To: Guillaume Le Vaillant; +Cc: help-guix, Sarah Morgensen
[-- Attachment #1: Type: text/plain, Size: 905 bytes --]
Hi Guillaume,
On Wednesday, September 29th, 2021 at 10:42 AM, Guillaume Le Vaillant <glv@posteo.net> wrote:
> Hi,
>
> I tried with simplified versions of your package definition and patch,
>
> and it builds fine for me. The patch gets applied successfully.
>
Thanks for the help and the package definition. I also managed to figure out what went wrong - I created the patch on master and attempted to patch 0.17.0.
The only other issue I had was with the xmlto, where I attempted to use local validation using docbook-xml based on example provided in gnu/packages/gnome.scm. See the attached file.
However, I gave up and also used the --disable-docs flag.
> However I haven't tried using the program to see if it works.
Well, that's another issue.
- It needs correct config file
- There has to be a service running. So I'll look into that now.
Thanks and have a nice day
Petr
[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: realmd.scm --]
[-- Type: text/x-scheme; name=realmd.scm, Size: 3034 bytes --]
(define-module (endgame packages realmd)
#:use-module (guix packages)
#:use-module (gnu packages)
#:use-module (gnu packages kerberos)
#:use-module (gnu packages openldap)
#:use-module (gnu packages glib)
#:use-module (gnu packages docbook)
#:use-module (gnu packages autotools)
#:use-module (gnu packages pkg-config)
#:use-module (gnu packages python)
#:use-module (gnu packages polkit)
#:use-module (gnu packages xml)
#:use-module (guix git-download)
#:use-module (guix build-system gnu)
#:use-module ((guix licenses) #:prefix license:))
(define-public realmd
(package
(name "realmd")
(version "0.17.0")
(source (origin
(method git-fetch)
(uri (git-reference
(url "https://github.com/freedesktop/realmd")
(commit version)))
(file-name (git-file-name name version))
(patches '("endgame/packages/patches/realmd-remove-distro-detection.patch"))
(sha256
(base32
"1c6q2a86kk2f1akzc36nh52hfwsmmc0mbp6ayyjxj4zsyk9zx5bf"))))
(build-system gnu-build-system)
(arguments
`(#:configure-flags (list "--with-systemd-unit-dir=no" "--with-systemd-journal=no" "--disable-doc")
#:phases
(modify-phases %standard-phases
(add-after 'unpack 'xml-validation
(lambda* (#:key inputs #:allow-other-keys)
(let ((local (string-append (assoc-ref inputs "docbook-xml") "/xml/dtd/docbook")))
; substitute in order to validate locally due to network restriction
(substitute* "doc/internals/realmd-internals.xml"
(("http://.*/xml/4.3") local))
(substitute* "doc/manual/realm.xml"
(("http://.*/xml/4.2") local))
(substitute* "doc/manual/realmd.conf.xml"
(("http://.*/xml/4.2") local))
(substitute* "doc/manual/realmd-docs.xml"
(("http://.*/xml/4.3") local))
(substitute* "doc/manual/realmd-guide-active-directory.xml"
(("http://.*/xml/4.3") local))
(substitute* "doc/manual/realmd-guide-ipa.xml"
(("http://.*/xml/4.3") local))
(substitute* "./doc/manual/realmd-guide-kerberos.xml"
(("http://.*/xml/4.3") local))#t))))))
(native-inputs `(("pkg-config" ,pkg-config)
("autoconf" ,autoconf)
("intltool" ,intltool)
("libxslt" ,libxslt)
("xmlto" ,xmlto)
("python" ,python)
("docbook-xml" ,docbook-xml)
("automake" ,automake)))
(inputs `(("glib" ,glib "bin")
("dbus" ,dbus)
("mit-krb5" ,mit-krb5)
("openldap" ,openldap)
("polkit" ,polkit)))
(synopsis "DBus service for configuring kerberos and other online identities")
(description "Dbus system service that manages discovery and enrollment
in realms/domains like Active Directory or IPA.")
(home-page "https://www.freedesktop.org/software/realmd/")
(license license:lgpl2.1)))
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: Apply patch to read-only file
@ 2021-09-28 4:47 Sarah Morgensen
0 siblings, 0 replies; 6+ messages in thread
From: Sarah Morgensen @ 2021-09-28 4:47 UTC (permalink / raw)
To: phodina; +Cc: help-guix
Hello Petr,
phodina <phodina@protonmail.com> writes:
> Hi Guillaume,
>
> Sure, here's the formatted file. My appology for the issue with the mail client.
>
> --8<---------------cut here---------------start------------->8---
>
> (define-module (realmd)
> #:use-module (guix packages)
> #:use-module (gnu packages)
> #:use-module (gnu packages image)
> #:use-module (gnu packages glib)
> #:use-module (gnu packages autotools)
> #:use-module (gnu packages pkg-config)
> #:use-module (gnu packages polkit)
> #:use-module (guix git-download)
> #:use-module (guix build-system gnu)
> #:use-module ((guix licenses) #:prefix license:))
>
> (define-public realmd
> (package
> (name "realmd")
> (version "0.17.0")
> (source (origin
> (method git-fetch)
> (uri (git-reference
> (url "https://github.com/freedesktop/realmd")
> (commit version)))
> (file-name (git-file-name name version))
> (modules '((guix build utils)))
> (snippet
> '(begin
> (delete-file "configure.ac")
> ;(make-file-writable "configure.ac")
> #t))
> (patches '("endgame/packages/patches/realmd-guixsd-support.patch"))
> (sha256
> (base32
> "1c6q2a86kk2f1akzc36nh52hfwsmmc0mbp6ayyjxj4zsyk9zx5bf"))))
> (build-system gnu-build-system)
> (arguments
> `(#:phases
> (modify-phases %standard-phases
> (add-after 'unpack 'make-git-checkout-writable
> (lambda* (#:key inputs #:allow-other-keys)
> (make-file-writable "configure.ac"))
> (add-after 'unpack 'apply-patches
> (for-each
> (lambda (file)
> (invoke "patch" "--force" "-p1" "-input" "endgame/packages/patches/realmd-guixsd-support.patch"))))
> ))))
>
> ; gnuzilla.scm
> ; jami.scm
> (native-inputs `(("pkg-config" ,pkg-config)
> ("autoconf" ,autoconf)
> ("intltool" ,intltool)
> ("automake" ,automake)))
> (inputs `(("glib" ,glib)
> ("polkit" ,polkit)))
> (synopsis "DBus service for configuring kerberos and other online identities")
> (description "Dbus system service that manages discovery and enrollment
> in realms/domains like Active Directory or IPA.")
> (home-page "https://www.freedesktop.org/software/realmd/")
> (license license:lgpl2.1)))
Is the issue maybe from deleting "configure.ac" in the snippet, then
trying to modify its permissions in 'make-git-checkout-writable'?
--
Sarah
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2021-10-02 19:49 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-09-16 7:49 Apply patch to read-only file phodina via
2021-09-25 12:08 ` Guillaume Le Vaillant
2021-09-27 22:57 ` phodina
2021-09-29 8:42 ` Guillaume Le Vaillant
2021-10-02 19:49 ` phodina
-- strict thread matches above, loose matches on Subject: below --
2021-09-28 4:47 Sarah Morgensen
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).