all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Marius Bakke <mbakke@fastmail.com>
To: Pierre Neidhardt <ambrevar@gmail.com>, 31999@debbugs.gnu.org
Subject: [bug#31999] [PATCH 5/7] gnu: Add dmraid.
Date: Mon, 30 Jul 2018 00:42:39 +0200	[thread overview]
Message-ID: <87muu93bq8.fsf@fastmail.com> (raw)
In-Reply-To: <20180728221001.14315-1-ambrevar@gmail.com>

[-- Attachment #1: Type: text/plain, Size: 2613 bytes --]

Pierre Neidhardt <ambrevar@gmail.com> writes:

> * gnu/package/disk.scm (dmraid): New variable.

[...]

> +(define-public dmraid
> +  (package
> +    (name "dmraid")
> +    (version "1.0.0.rc16-3")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "http://people.redhat.com/~heinzm/sw/dmraid/src/dmraid-"
> +                                  version
> +                                  ".tar.bz2"))

*HTTPS.  Also you can save a line here :-)

> +              (sha256
> +               (base32
> +                "1n7vsqvh7y6yvil682q129d21yhb0cmvd5fvsbkza7ypd78inhlk"))))

14 years since the last release and still going strong!

> +    (build-system gnu-build-system)
> +    (native-inputs
> +     `(("lvm2" ,lvm2)
> +       ("which" ,which)))

Is LVM2 really a native input?

> +    (arguments
> +     `(#:tests? #f                      ; No tests.
> +       #:phases (modify-phases %standard-phases
> +                  (add-before 'configure 'change-directory
> +                    (lambda _
> +                      (chdir (string-append ,version "/dmraid"))
> +                      (substitute* "make.tmpl.in"
> +                        (("/bin/sh") (which "sh")))
> +                      #t)))

Nit-pick: I prefer having the chdir just after unpack.  It's a huge
benefit on tarballs with lots of unrelated directories, so that the
various patching phases don't have to traverse everything.

I doubt it makes a difference in this case, though :P


> +       #:configure-flags (list ;; Make sure programs such as 'dmevent_tool' can
> +                               ;; find libdmraid.so.
> +                               (string-append "LDFLAGS=-Wl,-rpath="
> +                                              (assoc-ref %outputs "out")
> +                                              "/lib"))))
> +    (home-page "http://people.redhat.com/~heinzm/sw/dmraid/")

*HTTPS

> +    (synopsis "Device mapper RAID interface")
> +    (description
> +     "This software supports RAID device discovery, RAID set activation, creation,
> +removal, rebuild and display of properties for ATARAID/DDF1 metadata.
> +
> +@command{dmraid} uses @file{libdevmapper} and the device-mapper kernel runtime
> +to create devices with respective mappings for the ATARAID sets discovered.")
> +    (license license:gpl2+)))

OK!  I realized this is the "fake RAID" driver, apparently it's called
ATARAID.  It would be good to mention this is for specialized hardware,
and LVM itself can do "normal" RAID, but if you can't think of a good
way to squeeze it in don't sweat about it :-)

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 487 bytes --]

  reply	other threads:[~2018-07-29 22:43 UTC|newest]

Thread overview: 65+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2018-06-28 21:32 [bug#31999] [PATCH 1/7] gnu: Add volume-key Pierre Neidhardt
2018-06-28 21:35 ` [bug#31999] [PATCH 2/7] gnu: Add ndctl Pierre Neidhardt
2018-06-28 21:35   ` [bug#31999] [PATCH 3/7] gnu: Add libbytesize Pierre Neidhardt
2018-07-12 20:20     ` Marius Bakke
2018-07-28 22:07       ` Pierre Neidhardt
2018-07-29 22:10         ` Marius Bakke
2018-07-30  9:59           ` Pierre Neidhardt
2018-07-30 13:36             ` Pierre Neidhardt
2018-07-30 18:42               ` Marius Bakke
2018-06-28 21:35   ` [bug#31999] [PATCH 4/7] gnu: lvm2: Add device-mapper-event support Pierre Neidhardt
2018-07-12 21:26     ` Marius Bakke
2018-07-28 22:09       ` Pierre Neidhardt
2018-07-29 22:33         ` Marius Bakke
2018-07-30 10:10           ` Pierre Neidhardt
2018-07-30 10:12           ` Pierre Neidhardt
2018-07-30 12:50             ` Marius Bakke
2018-07-30 13:36               ` Pierre Neidhardt
2018-07-30 18:43                 ` Marius Bakke
2018-06-28 21:35   ` [bug#31999] [PATCH 5/7] gnu: Add dmraid Pierre Neidhardt
2018-07-12 21:28     ` Marius Bakke
2018-07-28 15:32       ` Pierre Neidhardt
2018-07-28 22:10         ` Pierre Neidhardt
2018-07-29 22:42           ` Marius Bakke [this message]
2018-07-30 10:23             ` Pierre Neidhardt
2018-07-30 13:37               ` Pierre Neidhardt
2018-07-30 18:44                 ` Marius Bakke
2018-08-01  3:21                   ` [bug#31999] [PATCH 5/7] gnu: Add dmraid. --- Failed to build from master Brendan Tildesley
2018-08-01 11:37                     ` Marius Bakke
2018-06-28 21:35   ` [bug#31999] [PATCH 6/7] gnu: Add libblockdev Pierre Neidhardt
2018-07-12 20:24     ` Marius Bakke
2018-07-28 22:10       ` Pierre Neidhardt
2018-07-29 22:56     ` Marius Bakke
2018-07-30  9:14       ` Pierre Neidhardt
2018-07-30 12:52         ` Marius Bakke
2018-07-30 13:37           ` Pierre Neidhardt
2018-07-30 18:48             ` Marius Bakke
2018-06-28 21:35   ` [bug#31999] [PATCH 7/7] gnu: Update udisk to 2.7.6 Pierre Neidhardt
2018-06-28 21:40     ` Pierre Neidhardt
2018-07-12 21:34     ` Marius Bakke
2018-07-28 22:11       ` Pierre Neidhardt
2018-07-28 22:17         ` Pierre Neidhardt
2018-07-29 23:01         ` Marius Bakke
2018-07-30 13:38           ` [bug#31999] [PATCH 7/7] gnu: Update udisks to 2.7.7 Pierre Neidhardt
2018-07-30 18:54             ` Marius Bakke
2018-07-30 19:20               ` Pierre Neidhardt
2018-07-30 19:36                 ` Pierre Neidhardt
2018-07-30 19:39                 ` Marius Bakke
2018-07-30 19:50                   ` Pierre Neidhardt
2018-07-30 20:59                     ` Marius Bakke
2018-07-12 20:12   ` [bug#31999] [PATCH 2/7] gnu: Add ndctl Marius Bakke
2018-07-28 22:05     ` Pierre Neidhardt
2018-07-29 22:23       ` Marius Bakke
2018-07-30  9:47         ` Pierre Neidhardt
2018-07-30 12:54           ` Marius Bakke
2018-07-30 13:35             ` Pierre Neidhardt
2018-07-30 18:56               ` Marius Bakke
2018-07-12 20:04 ` [bug#31999] [PATCH 1/7] gnu: Add volume-key Marius Bakke
2018-07-28 21:58   ` [bug#32299] " Pierre Neidhardt
2018-07-28 22:02     ` bug#32299: " Pierre Neidhardt
2018-07-29 22:28     ` [bug#32299] " Marius Bakke
2018-07-30  9:19       ` [bug#31999] " Pierre Neidhardt
2018-07-30 13:16         ` [bug#31999] " Pierre Neidhardt
2018-07-30 19:01           ` Marius Bakke
2018-07-28 22:04   ` Pierre Neidhardt
2018-07-30 19:53 ` bug#31999: [PATCH 7/7] gnu: Update udisks to 2.7.7 Pierre Neidhardt

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=87muu93bq8.fsf@fastmail.com \
    --to=mbakke@fastmail.com \
    --cc=31999@debbugs.gnu.org \
    --cc=ambrevar@gmail.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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

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.