unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH v2] gnu: Add mtd-utils.
@ 2016-09-01 12:33 Danny Milosavljevic
  2016-09-06 21:49 ` Leo Famulari
  0 siblings, 1 reply; 6+ messages in thread
From: Danny Milosavljevic @ 2016-09-01 12:33 UTC (permalink / raw)
  To: guix-devel

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

gnu: Add mtd-utils.

* gnu/packages/linux.scm (mtd-utils): New variable.
---
 gnu/packages/linux.scm | 36 ++++++++++++++++++++++++++++++++++++
 1 file changed, 36 insertions(+)


[-- Warning: decoded text below may be mangled, UTF-8 assumed --]
[-- Attachment #2: 0001-gnu-Add-mtd-utils.patch --]
[-- Type: text/x-patch; name="0001-gnu-Add-mtd-utils.patch", Size: 1764 bytes --]

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index aac2cd1..4c690a6 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -2924,3 +2924,39 @@ native Linux file system, and has been part of the Linux kernel since version
     ;; The files src/key_mod/ecryptfs_key_mod_{openssl,pkcs11_helper,tspi}.c
     ;; grant additional permission to link with OpenSSL.
     (license license:gpl2+)))
+
+(define-public mtd-utils
+  (package
+    (name "mtd-utils")
+    (version "1.5.2")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "ftp://ftp.infradead.org/pub/mtd-utils/"
+                    "mtd-utils-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "007lhsd8yb34l899r4m37whhzdw815cz4fnjbpnblfha524p7dax"))))
+    (inputs
+     `(("acl" ,acl)
+       ("libuuid" ,util-linux)
+       ("lzo", lzo)
+       ("zlib" ,zlib)))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:test-target "tests"
+       #:phases (modify-phases %standard-phases
+         (add-before 'build 'patch-installation-prefix
+           (lambda* (#:key outputs #:allow-other-keys)
+             (let ((out (assoc-ref outputs "out")))
+               (substitute* '("common.mk")
+                 (("PREFIX=/usr") (string-append "PREFIX=" out)))
+               #t)))
+         (delete 'configure))))
+    (synopsis "MTD Flash Storage Utilities")
+    (description
+     "This package provides utilities for testing, partitioning
+etc of flash storage.")
+    (home-page "http://www.linux-mtd.infradead.org/")
+    ;; mkfs.ubifs/crc16.h is GPLv2
+    (license (list license:gpl2 license:mpl1.1 license:bsd-3))))

^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] gnu: Add mtd-utils.
  2016-09-01 12:33 [PATCH v2] gnu: Add mtd-utils Danny Milosavljevic
@ 2016-09-06 21:49 ` Leo Famulari
  2016-09-07  6:29   ` Efraim Flashner
  2016-09-07  7:27   ` Danny Milosavljevic
  0 siblings, 2 replies; 6+ messages in thread
From: Leo Famulari @ 2016-09-06 21:49 UTC (permalink / raw)
  To: Danny Milosavljevic; +Cc: guix-devel

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

On Thu, Sep 01, 2016 at 02:33:57PM +0200, Danny Milosavljevic wrote:
> gnu: Add mtd-utils.
> 
> * gnu/packages/linux.scm (mtd-utils): New variable.

Thanks for this patch!

I made some adjustments, as attached.

I imported the acl module, used #:make-flags to set PREFIX, and made a
minor adjustment to the description.

What do you think?

[-- Attachment #2: 0001-gnu-Add-mtd-utils.patch --]
[-- Type: text/plain, Size: 2152 bytes --]

From dd00ffc45982b393cb458445e875188baf77175f Mon Sep 17 00:00:00 2001
From: Danny Milosavljevic <dannym@scratchpost.org>
Date: Thu, 1 Sep 2016 14:33:57 +0200
Subject: [PATCH] gnu: Add mtd-utils.

* gnu/packages/linux.scm (mtd-utils): New variable.

Signed-off-by: Leo Famulari <leo@famulari.name>
---
 gnu/packages/linux.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index 030e35e..ed96996 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -33,6 +33,7 @@
 
 (define-module (gnu packages linux)
   #:use-module (gnu packages)
+  #:use-module (gnu packages acl)
   #:use-module (gnu packages admin)
   #:use-module (gnu packages algebra)
   #:use-module (gnu packages attr)
@@ -2929,3 +2930,33 @@ native Linux file system, and has been part of the Linux kernel since version
     ;; The files src/key_mod/ecryptfs_key_mod_{openssl,pkcs11_helper,tspi}.c
     ;; grant additional permission to link with OpenSSL.
     (license license:gpl2+)))
+
+(define-public mtd-utils
+  (package
+    (name "mtd-utils")
+    (version "1.5.2")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append
+                    "ftp://ftp.infradead.org/pub/mtd-utils/"
+                    "mtd-utils-" version ".tar.bz2"))
+              (sha256
+               (base32
+                "007lhsd8yb34l899r4m37whhzdw815cz4fnjbpnblfha524p7dax"))))
+    (inputs
+     `(("acl" ,acl)
+       ("libuuid" ,util-linux)
+       ("lzo", lzo)
+       ("zlib" ,zlib)))
+    (build-system gnu-build-system)
+    (arguments
+     `(#:test-target "tests"
+       #:make-flags (list (string-append "PREFIX=" (assoc-ref %outputs "out")))
+       #:phases (modify-phases %standard-phases
+                  (delete 'configure))))
+    (synopsis "MTD Flash Storage Utilities")
+    (description "This package provides utilities for testing, partitioning, etc
+of flash storage.")
+    (home-page "http://www.linux-mtd.infradead.org/")
+    ;; mkfs.ubifs/crc16.h is GPLv2
+    (license (list license:gpl2 license:mpl1.1 license:bsd-3))))
-- 
2.10.0


^ permalink raw reply related	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] gnu: Add mtd-utils.
  2016-09-06 21:49 ` Leo Famulari
@ 2016-09-07  6:29   ` Efraim Flashner
  2016-09-10 19:45     ` Leo Famulari
  2016-09-07  7:27   ` Danny Milosavljevic
  1 sibling, 1 reply; 6+ messages in thread
From: Efraim Flashner @ 2016-09-07  6:29 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

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

On Tue, Sep 06, 2016 at 05:49:41PM -0400, Leo Famulari wrote:
> On Thu, Sep 01, 2016 at 02:33:57PM +0200, Danny Milosavljevic wrote:
> > gnu: Add mtd-utils.
> > 
> > * gnu/packages/linux.scm (mtd-utils): New variable.
> 
> Thanks for this patch!
> 
> I made some adjustments, as attached.
> 
> I imported the acl module, used #:make-flags to set PREFIX, and made a
> minor adjustment to the description.
> 
> What do you think?

> From dd00ffc45982b393cb458445e875188baf77175f Mon Sep 17 00:00:00 2001
> From: Danny Milosavljevic <dannym@scratchpost.org>
> Date: Thu, 1 Sep 2016 14:33:57 +0200
> Subject: [PATCH] gnu: Add mtd-utils.
> 
> * gnu/packages/linux.scm (mtd-utils): New variable.
> 
> Signed-off-by: Leo Famulari <leo@famulari.name>
> ---
>  gnu/packages/linux.scm | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
> 
> diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> index 030e35e..ed96996 100644
> --- a/gnu/packages/linux.scm
> +++ b/gnu/packages/linux.scm
> @@ -33,6 +33,7 @@
>  
>  (define-module (gnu packages linux)
>    #:use-module (gnu packages)
> +  #:use-module (gnu packages acl)
>    #:use-module (gnu packages admin)
>    #:use-module (gnu packages algebra)
>    #:use-module (gnu packages attr)
> @@ -2929,3 +2930,33 @@ native Linux file system, and has been part of the Linux kernel since version
>      ;; The files src/key_mod/ecryptfs_key_mod_{openssl,pkcs11_helper,tspi}.c
>      ;; grant additional permission to link with OpenSSL.
>      (license license:gpl2+)))
> +
> +(define-public mtd-utils
> +  (package
> +    (name "mtd-utils")
> +    (version "1.5.2")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append
> +                    "ftp://ftp.infradead.org/pub/mtd-utils/"
> +                    "mtd-utils-" version ".tar.bz2"))
> +              (sha256
> +               (base32
> +                "007lhsd8yb34l899r4m37whhzdw815cz4fnjbpnblfha524p7dax"))))
> +    (inputs
> +     `(("acl" ,acl)
> +       ("libuuid" ,util-linux)
> +       ("lzo", lzo)
> +       ("zlib" ,zlib)))
> +    (build-system gnu-build-system)
> +    (arguments
> +     `(#:test-target "tests"
> +       #:make-flags (list (string-append "PREFIX=" (assoc-ref %outputs "out")))
> +       #:phases (modify-phases %standard-phases
> +                  (delete 'configure))))
> +    (synopsis "MTD Flash Storage Utilities")
> +    (description "This package provides utilities for testing, partitioning, etc
> +of flash storage.")
> +    (home-page "http://www.linux-mtd.infradead.org/")
> +    ;; mkfs.ubifs/crc16.h is GPLv2
> +    (license (list license:gpl2 license:mpl1.1 license:bsd-3))))
> -- 
> 2.10.0
> 

Is the package under all three licenses or are certain parts under
different licenses? I've found packages that include a mozilla license
seem to have these three, or a variation of them.


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] gnu: Add mtd-utils.
  2016-09-06 21:49 ` Leo Famulari
  2016-09-07  6:29   ` Efraim Flashner
@ 2016-09-07  7:27   ` Danny Milosavljevic
  1 sibling, 0 replies; 6+ messages in thread
From: Danny Milosavljevic @ 2016-09-07  7:27 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Hi Leo,

On Tue, 6 Sep 2016 17:49:41 -0400
Leo Famulari <leo@famulari.name> wrote:

> I imported the acl module, used #:make-flags to set PREFIX, and made a
> minor adjustment to the description.
> 
> What do you think?

Looks good and works :)

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] gnu: Add mtd-utils.
  2016-09-07  6:29   ` Efraim Flashner
@ 2016-09-10 19:45     ` Leo Famulari
  2016-09-26  9:39       ` Efraim Flashner
  0 siblings, 1 reply; 6+ messages in thread
From: Leo Famulari @ 2016-09-10 19:45 UTC (permalink / raw)
  To: Efraim Flashner; +Cc: guix-devel

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

On Wed, Sep 07, 2016 at 09:29:46AM +0300, Efraim Flashner wrote:
> > +    (license (list license:gpl2 license:mpl1.1 license:bsd-3))))
> 
> Is the package under all three licenses or are certain parts under
> different licenses? I've found packages that include a mozilla license
> seem to have these three, or a variation of them.

Ping :)

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

* Re: [PATCH v2] gnu: Add mtd-utils.
  2016-09-10 19:45     ` Leo Famulari
@ 2016-09-26  9:39       ` Efraim Flashner
  0 siblings, 0 replies; 6+ messages in thread
From: Efraim Flashner @ 2016-09-26  9:39 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

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

On Sat, Sep 10, 2016 at 03:45:16PM -0400, Leo Famulari wrote:
> On Wed, Sep 07, 2016 at 09:29:46AM +0300, Efraim Flashner wrote:
> > > +    (license (list license:gpl2 license:mpl1.1 license:bsd-3))))
> > 
> > Is the package under all three licenses or are certain parts under
> > different licenses? I've found packages that include a mozilla license
> > seem to have these three, or a variation of them.
> 
> Ping :)

After grepping License, here's what I got:

jffsdump.c        gpl2
compr_lzo.c       gpl2
mtdpart.c         gpl2
ftl_format.c      mpl1.1
ftl_check.c       mpl1.1
./ubi-utils/LICENSE.libiniparser:MIT License
tests/ubi-tests   gpl2+
tests/fs-tests    gpl2
mkfs.ubifs/       gpl2+
include/mtd/ftl-user.h  mpl1.1

everything else in include/ and loose files in / are gpl2+


-- 
Efraim Flashner   <efraim@flashner.co.il>   אפרים פלשנר
GPG key = A28B F40C 3E55 1372 662D  14F7 41AA E7DC CA3D 8351
Confidentiality cannot be guaranteed on emails sent or received unencrypted

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

^ permalink raw reply	[flat|nested] 6+ messages in thread

end of thread, other threads:[~2016-09-26  9:39 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-09-01 12:33 [PATCH v2] gnu: Add mtd-utils Danny Milosavljevic
2016-09-06 21:49 ` Leo Famulari
2016-09-07  6:29   ` Efraim Flashner
2016-09-10 19:45     ` Leo Famulari
2016-09-26  9:39       ` Efraim Flashner
2016-09-07  7:27   ` Danny Milosavljevic

Code repositories for project(s) associated with this public inbox

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

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).