all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* bug#27073: [PATCH] gnu: Add cpuid.
@ 2017-05-25 18:23 Clément Lassieur
  2017-05-25 18:36 ` Clément Lassieur
  2017-05-25 18:54 ` Marius Bakke
  0 siblings, 2 replies; 6+ messages in thread
From: Clément Lassieur @ 2017-05-25 18:23 UTC (permalink / raw)
  To: 27073

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

diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
index 2e984dd42..d888ec2bb 100644
--- a/gnu/packages/linux.scm
+++ b/gnu/packages/linux.scm
@@ -23,6 +23,7 @@
 ;;; Copyright © 2017 José Miguel Sánchez García <jmi2k@openmailbox.com>
 ;;; Copyright © 2017 Gábor Boskovits <boskovits@gmail.com>
 ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
+;;; Copyright © 2017 Clément Lassieur <clement@lassieur.org>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -3948,3 +3949,37 @@ available in the kernel Linux.")
                    #t))))))
        ((#:allowed-references _ '("out"))
         '("out"))))))
+
+(define-public cpuid
+  (package
+    (name "cpuid")
+    (version "20170122")
+    (source (origin
+              (method url-fetch)
+              (uri (string-append "http://www.etallen.com/cpuid/cpuid-"
+                                  version ".src.tar.gz"))
+              (sha256
+               (base32
+                "0ra8ph9m1dckqaikfnbsh408fp2w9k49fkl423fl2hvhwsm14xk6"))))
+    (build-system gnu-build-system)
+    (arguments
+     '(#:make-flags '("CC=gcc")
+       #:tests? #f ; no tests
+       #:phases (modify-phases %standard-phases
+                  (delete 'configure)
+                  (add-before 'install 'fix-makefile
+                    (lambda* (#:key outputs #:allow-other-keys)
+                      (substitute* "Makefile"
+                        (("\\$\\(BUILDROOT\\)/usr") (assoc-ref outputs "out")))
+                      (substitute* "Makefile"
+                        (("-m 444") "-m 644"))
+                      #t)))))
+    (inputs `(("perl" ,perl)))
+    (supported-systems '("i686-linux" "x86_64-linux"))
+    (home-page "http://etallen.com/cpuid.html")
+    (synopsis "Linux tool to dump x86 CPUID information about the CPU(s)")
+    (description "cpuid dumps detailed information about the CPU(s) gathered
+from the CPUID instruction, and also determines the exact model of CPU(s).  It
+supports Intel, AMD, and VIA CPUs, as well as older Transmeta, Cyrix, UMC,
+NexGen, Rise, and SiS CPUs.")
+    (license license:gpl2+)))
-- 
2.13.0

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

* bug#27073: [PATCH] gnu: Add cpuid.
  2017-05-25 18:23 bug#27073: [PATCH] gnu: Add cpuid Clément Lassieur
@ 2017-05-25 18:36 ` Clément Lassieur
  2017-05-25 18:41   ` ng0
  2017-05-25 18:54 ` Marius Bakke
  1 sibling, 1 reply; 6+ messages in thread
From: Clément Lassieur @ 2017-05-25 18:36 UTC (permalink / raw)
  To: 27073

Clément Lassieur <clement@lassieur.org> writes:

> * gnu/packages/linux.scm (cpuid): New variable.
> ---
>  gnu/packages/linux.scm | 35 +++++++++++++++++++++++++++++++++++
>  1 file changed, 35 insertions(+)
>
> diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> index 2e984dd42..d888ec2bb 100644
> --- a/gnu/packages/linux.scm
> +++ b/gnu/packages/linux.scm
> @@ -23,6 +23,7 @@
>  ;;; Copyright © 2017 José Miguel Sánchez García <jmi2k@openmailbox.com>
>  ;;; Copyright © 2017 Gábor Boskovits <boskovits@gmail.com>
>  ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
> +;;; Copyright © 2017 Clément Lassieur <clement@lassieur.org>
>  ;;;
>  ;;; This file is part of GNU Guix.
>  ;;;
> @@ -3948,3 +3949,37 @@ available in the kernel Linux.")
>                     #t))))))
>         ((#:allowed-references _ '("out"))
>          '("out"))))))
> +
> +(define-public cpuid
> +  (package
> +    (name "cpuid")
> +    (version "20170122")
> +    (source (origin
> +              (method url-fetch)
> +              (uri (string-append "http://www.etallen.com/cpuid/cpuid-"
> +                                  version ".src.tar.gz"))
> +              (sha256
> +               (base32
> +                "0ra8ph9m1dckqaikfnbsh408fp2w9k49fkl423fl2hvhwsm14xk6"))))
> +    (build-system gnu-build-system)
> +    (arguments
> +     '(#:make-flags '("CC=gcc")
> +       #:tests? #f ; no tests
> +       #:phases (modify-phases %standard-phases
> +                  (delete 'configure)
> +                  (add-before 'install 'fix-makefile
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (substitute* "Makefile"
> +                        (("\\$\\(BUILDROOT\\)/usr") (assoc-ref outputs "out")))
> +                      (substitute* "Makefile"
> +                        (("-m 444") "-m 644"))
> +                      #t)))))
> +    (inputs `(("perl" ,perl)))
> +    (supported-systems '("i686-linux" "x86_64-linux"))
> +    (home-page "http://etallen.com/cpuid.html")

with "www." here: --------^

> +    (synopsis "Linux tool to dump x86 CPUID information about the CPU(s)")
> +    (description "cpuid dumps detailed information about the CPU(s) gathered
> +from the CPUID instruction, and also determines the exact model of CPU(s).  It
> +supports Intel, AMD, and VIA CPUs, as well as older Transmeta, Cyrix, UMC,
> +NexGen, Rise, and SiS CPUs.")
> +    (license license:gpl2+)))

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

* bug#27073: [PATCH] gnu: Add cpuid.
  2017-05-25 18:36 ` Clément Lassieur
@ 2017-05-25 18:41   ` ng0
  2017-05-25 19:13     ` Clément Lassieur
  0 siblings, 1 reply; 6+ messages in thread
From: ng0 @ 2017-05-25 18:41 UTC (permalink / raw)
  To: 27073



On Thu, 25 May 2017 20:36:23 +0200, Clément Lassieur <clement@lassieur.org> wrote:

> Clément Lassieur <clement@lassieur.org> writes:
> 
> > * gnu/packages/linux.scm (cpuid): New variable.
> > ---
> >  gnu/packages/linux.scm | 35 +++++++++++++++++++++++++++++++++++
> >  1 file changed, 35 insertions(+)
> >
> > diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
> > index 2e984dd42..d888ec2bb 100644
> > --- a/gnu/packages/linux.scm
> > +++ b/gnu/packages/linux.scm
> > @@ -23,6 +23,7 @@
> >  ;;; Copyright © 2017 José Miguel Sánchez García <jmi2k@openmailbox.com>
> >  ;;; Copyright © 2017 Gábor Boskovits <boskovits@gmail.com>
> >  ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
> > +;;; Copyright © 2017 Clément Lassieur <clement@lassieur.org>
> >  ;;;
> >  ;;; This file is part of GNU Guix.
> >  ;;;
> > @@ -3948,3 +3949,37 @@ available in the kernel Linux.")
> >                     #t))))))
> >         ((#:allowed-references _ '("out"))
> >          '("out"))))))
> > +
> > +(define-public cpuid
> > +  (package
> > +    (name "cpuid")
> > +    (version "20170122")
> > +    (source (origin
> > +              (method url-fetch)
> > +              (uri (string-append "http://www.etallen.com/cpuid/cpuid-"
> > +                                  version ".src.tar.gz"))
> > +              (sha256
> > +               (base32
> > +                "0ra8ph9m1dckqaikfnbsh408fp2w9k49fkl423fl2hvhwsm14xk6"))))
> > +    (build-system gnu-build-system)
> > +    (arguments
> > +     '(#:make-flags '("CC=gcc")
> > +       #:tests? #f ; no tests
> > +       #:phases (modify-phases %standard-phases
> > +                  (delete 'configure)
> > +                  (add-before 'install 'fix-makefile
> > +                    (lambda* (#:key outputs #:allow-other-keys)
> > +                      (substitute* "Makefile"
> > +                        (("\\$\\(BUILDROOT\\)/usr") (assoc-ref outputs "out")))
> > +                      (substitute* "Makefile"
> > +                        (("-m 444") "-m 644"))
> > +                      #t)))))
> > +    (inputs `(("perl" ,perl)))
> > +    (supported-systems '("i686-linux" "x86_64-linux"))
> > +    (home-page "http://etallen.com/cpuid.html")
> 
> with "www." here: --------^

Actually it doesn't matter. The site can be accessed at www.* and without www.*

> 
> > +    (synopsis "Linux tool to dump x86 CPUID information about the CPU(s)")
> > +    (description "cpuid dumps detailed information about the CPU(s) gathered
> > +from the CPUID instruction, and also determines the exact model of CPU(s).  It
> > +supports Intel, AMD, and VIA CPUs, as well as older Transmeta, Cyrix, UMC,
> > +NexGen, Rise, and SiS CPUs.")
> > +    (license license:gpl2+)))

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

* bug#27073: [PATCH] gnu: Add cpuid.
  2017-05-25 18:23 bug#27073: [PATCH] gnu: Add cpuid Clément Lassieur
  2017-05-25 18:36 ` Clément Lassieur
@ 2017-05-25 18:54 ` Marius Bakke
  2017-05-25 19:41   ` Clément Lassieur
  1 sibling, 1 reply; 6+ messages in thread
From: Marius Bakke @ 2017-05-25 18:54 UTC (permalink / raw)
  To: Clément Lassieur, 27073

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

Clément Lassieur <clement@lassieur.org> writes:

> * gnu/packages/linux.scm (cpuid): New variable.

[...]

> +       #:phases (modify-phases %standard-phases
> +                  (delete 'configure)
> +                  (add-before 'install 'fix-makefile
> +                    (lambda* (#:key outputs #:allow-other-keys)
> +                      (substitute* "Makefile"
> +                        (("\\$\\(BUILDROOT\\)/usr") (assoc-ref outputs "out")))
> +                      (substitute* "Makefile"
> +                        (("-m 444") "-m 644"))

Why is this necessary? Perhaps you can add a comment?

LGTM otherwise.

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

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

* bug#27073: [PATCH] gnu: Add cpuid.
  2017-05-25 18:41   ` ng0
@ 2017-05-25 19:13     ` Clément Lassieur
  0 siblings, 0 replies; 6+ messages in thread
From: Clément Lassieur @ 2017-05-25 19:13 UTC (permalink / raw)
  To: ng0; +Cc: 27073

ng0@pragmatique.xyz writes:

> On Thu, 25 May 2017 20:36:23 +0200, Clément Lassieur <clement@lassieur.org> wrote:
>
>> Clément Lassieur <clement@lassieur.org> writes:
>> 
>> > * gnu/packages/linux.scm (cpuid): New variable.
>> > ---
>> >  gnu/packages/linux.scm | 35 +++++++++++++++++++++++++++++++++++
>> >  1 file changed, 35 insertions(+)
>> >
>> > diff --git a/gnu/packages/linux.scm b/gnu/packages/linux.scm
>> > index 2e984dd42..d888ec2bb 100644
>> > --- a/gnu/packages/linux.scm
>> > +++ b/gnu/packages/linux.scm
>> > @@ -23,6 +23,7 @@
>> >  ;;; Copyright © 2017 José Miguel Sánchez García <jmi2k@openmailbox.com>
>> >  ;;; Copyright © 2017 Gábor Boskovits <boskovits@gmail.com>
>> >  ;;; Copyright © 2017 Mathieu Othacehe <m.othacehe@gmail.com>
>> > +;;; Copyright © 2017 Clément Lassieur <clement@lassieur.org>
>> >  ;;;
>> >  ;;; This file is part of GNU Guix.
>> >  ;;;
>> > @@ -3948,3 +3949,37 @@ available in the kernel Linux.")
>> >                     #t))))))
>> >         ((#:allowed-references _ '("out"))
>> >          '("out"))))))
>> > +
>> > +(define-public cpuid
>> > +  (package
>> > +    (name "cpuid")
>> > +    (version "20170122")
>> > +    (source (origin
>> > +              (method url-fetch)
>> > +              (uri (string-append "http://www.etallen.com/cpuid/cpuid-"
>> > +                                  version ".src.tar.gz"))
>> > +              (sha256
>> > +               (base32
>> > +                "0ra8ph9m1dckqaikfnbsh408fp2w9k49fkl423fl2hvhwsm14xk6"))))
>> > +    (build-system gnu-build-system)
>> > +    (arguments
>> > +     '(#:make-flags '("CC=gcc")
>> > +       #:tests? #f ; no tests
>> > +       #:phases (modify-phases %standard-phases
>> > +                  (delete 'configure)
>> > +                  (add-before 'install 'fix-makefile
>> > +                    (lambda* (#:key outputs #:allow-other-keys)
>> > +                      (substitute* "Makefile"
>> > +                        (("\\$\\(BUILDROOT\\)/usr") (assoc-ref outputs "out")))
>> > +                      (substitute* "Makefile"
>> > +                        (("-m 444") "-m 644"))
>> > +                      #t)))))
>> > +    (inputs `(("perl" ,perl)))
>> > +    (supported-systems '("i686-linux" "x86_64-linux"))
>> > +    (home-page "http://etallen.com/cpuid.html")
>> 
>> with "www." here: --------^
>
> Actually it doesn't matter. The site can be accessed at www.* and without www.*

Yes, I added it so it matches with 'source' :-)
>
>> 
>> > +    (synopsis "Linux tool to dump x86 CPUID information about the CPU(s)")
>> > +    (description "cpuid dumps detailed information about the CPU(s) gathered
>> > +from the CPUID instruction, and also determines the exact model of CPU(s).  It
>> > +supports Intel, AMD, and VIA CPUs, as well as older Transmeta, Cyrix, UMC,
>> > +NexGen, Rise, and SiS CPUs.")
>> > +    (license license:gpl2+)))

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

* bug#27073: [PATCH] gnu: Add cpuid.
  2017-05-25 18:54 ` Marius Bakke
@ 2017-05-25 19:41   ` Clément Lassieur
  0 siblings, 0 replies; 6+ messages in thread
From: Clément Lassieur @ 2017-05-25 19:41 UTC (permalink / raw)
  To: Marius Bakke; +Cc: 27073-done

Marius Bakke <mbakke@fastmail.com> writes:

> Clément Lassieur <clement@lassieur.org> writes:
>
>> * gnu/packages/linux.scm (cpuid): New variable.
>
> [...]
>
>> +       #:phases (modify-phases %standard-phases
>> +                  (delete 'configure)
>> +                  (add-before 'install 'fix-makefile
>> +                    (lambda* (#:key outputs #:allow-other-keys)
>> +                      (substitute* "Makefile"
>> +                        (("\\$\\(BUILDROOT\\)/usr") (assoc-ref outputs "out")))
>> +                      (substitute* "Makefile"
>> +                        (("-m 444") "-m 644"))
>
> Why is this necessary? Perhaps you can add a comment?

I added:

;; Make the compressed manpages writable so that the
;; reset-gzip-timestamps phase does not error out.

> LGTM otherwise.

Thanks for the review :-) Pushed.

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

end of thread, other threads:[~2017-05-25 19:42 UTC | newest]

Thread overview: 6+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-05-25 18:23 bug#27073: [PATCH] gnu: Add cpuid Clément Lassieur
2017-05-25 18:36 ` Clément Lassieur
2017-05-25 18:41   ` ng0
2017-05-25 19:13     ` Clément Lassieur
2017-05-25 18:54 ` Marius Bakke
2017-05-25 19:41   ` Clément Lassieur

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.