* [bug#40902] [PATCH] gnu: Add edid-decode.
@ 2020-04-27 13:55 Brice Waegeneire
2020-04-28 7:46 ` Mathieu Othacehe
2020-04-28 9:53 ` [bug#40902] [PATCH v2] " Brice Waegeneire
0 siblings, 2 replies; 5+ messages in thread
From: Brice Waegeneire @ 2020-04-27 13:55 UTC (permalink / raw)
To: 40902
* gnu/packages/hardware.scm (edid-decode): New variable.
---
gnu/packages/hardware.scm | 30 ++++++++++++++++++++++++++++++
1 file changed, 30 insertions(+)
diff --git a/gnu/packages/hardware.scm b/gnu/packages/hardware.scm
index 189c73ee3c..3284733095 100644
--- a/gnu/packages/hardware.scm
+++ b/gnu/packages/hardware.scm
@@ -89,6 +89,36 @@ calibrated, and restored when the calibration is applied.")
(license (list license:bsd-3 ; FindDDCUtil.cmake
license:gpl2+)))) ; everything else
+(define-public edid-decode
+ (let ((commit "74b64180d67bb009d8d9ea1b6f18ad41aaa16396") ; 2020-04-22
+ (revision "1"))
+ (package
+ (name "edid-decode")
+ (version (git-version "0.0.0" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (file-name (git-file-name name version))
+ (uri (git-reference
+ (url "git://linuxtv.org/edid-decode.git")
+ (commit commit)))
+ (sha256
+ (base32 "0nirp5bza08zj5d8bjgcm0p869hdg3qg3mwa7999pjdrzmn7s2ah"))))
+ (build-system gnu-build-system)
+ (arguments
+ `(#:tests? #f ; No test suite
+ #:make-flags
+ (list (string-append "DESTDIR=" (assoc-ref %outputs "out"))
+ "bindir=/bin" "mandir=/share/man")
+ #:phases
+ (modify-phases %standard-phases
+ (delete 'configure))))
+ (home-page "https://git.linuxtv.org/edid-decode.git/")
+ (synopsis "Decode @dfn{EDID} data in human-readable format")
+ (description "edid-decode decodes @dfn{EDID} monitor description data in
+human-readable format and check if it conforms to the standards.")
+ (license license:expat))))
+
;; Distinct from memtest86, which is obsolete.
(define-public memtest86+
(package
--
2.26.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* [bug#40902] [PATCH] gnu: Add edid-decode.
2020-04-27 13:55 [bug#40902] [PATCH] gnu: Add edid-decode Brice Waegeneire
@ 2020-04-28 7:46 ` Mathieu Othacehe
2020-04-28 9:54 ` Brice Waegeneire
2020-04-28 9:53 ` [bug#40902] [PATCH v2] " Brice Waegeneire
1 sibling, 1 reply; 5+ messages in thread
From: Mathieu Othacehe @ 2020-04-28 7:46 UTC (permalink / raw)
To: Brice Waegeneire; +Cc: 40902
Hello Brice,
You can add your copyright.
> +(define-public edid-decode
> + (let ((commit "74b64180d67bb009d8d9ea1b6f18ad41aaa16396") ; 2020-04-22
> + (revision "1"))
> + (package
> + (name "edid-decode")
> + (version (git-version "0.0.0" revision commit))
The indentation of the block seems wrong.
> + (source
> + (origin
> + (method git-fetch)
> + (file-name (git-file-name name version))
> + (uri (git-reference
> + (url "git://linuxtv.org/edid-decode.git")
> + (commit commit)))
> + (sha256
> + (base32 "0nirp5bza08zj5d8bjgcm0p869hdg3qg3mwa7999pjdrzmn7s2ah"))))
> + (build-system gnu-build-system)
> + (arguments
> + `(#:tests? #f ; No test suite
> + #:make-flags
> + (list (string-append "DESTDIR=" (assoc-ref %outputs "out"))
> + "bindir=/bin" "mandir=/share/man")
> + #:phases
> + (modify-phases %standard-phases
> + (delete 'configure))))
When cross-compiling, a native binary is produced. This is because CXX
is set to 'g++' I guess. You need to do something similar to what is
done in the 'maradns' package.
Thanks,
Mathieu
^ permalink raw reply [flat|nested] 5+ messages in thread
* [bug#40902] [PATCH] gnu: Add edid-decode.
2020-04-28 7:46 ` Mathieu Othacehe
@ 2020-04-28 9:54 ` Brice Waegeneire
0 siblings, 0 replies; 5+ messages in thread
From: Brice Waegeneire @ 2020-04-28 9:54 UTC (permalink / raw)
To: Mathieu Othacehe; +Cc: 40902
Hello Mathieu,
Thank you for taking the time to review this patch.
On 2020-04-28 07:46, Mathieu Othacehe wrote:
> Hello Brice,
>
> You can add your copyright.
I already added it in #40607[0], an unmerged patch adding usbguard.
>> +(define-public edid-decode
>> + (let ((commit "74b64180d67bb009d8d9ea1b6f18ad41aaa16396") ;
>> 2020-04-22
>> + (revision "1"))
>> + (package
>> + (name "edid-decode")
>> + (version (git-version "0.0.0" revision commit))
>
> The indentation of the block seems wrong.
You are correct, “guix lint” didn't mentioned it though. Fixed in v2.
>> + (source
>> + (origin
>> + (method git-fetch)
>> + (file-name (git-file-name name version))
>> + (uri (git-reference
>> + (url "git://linuxtv.org/edid-decode.git")
>> + (commit commit)))
>> + (sha256
>> + (base32
>> "0nirp5bza08zj5d8bjgcm0p869hdg3qg3mwa7999pjdrzmn7s2ah"))))
>> + (build-system gnu-build-system)
>> + (arguments
>> + `(#:tests? #f ; No test suite
>> + #:make-flags
>> + (list (string-append "DESTDIR=" (assoc-ref %outputs "out"))
>> + "bindir=/bin" "mandir=/share/man")
>> + #:phases
>> + (modify-phases %standard-phases
>> + (delete 'configure))))
>
> When cross-compiling, a native binary is produced. This is because CXX
> is set to 'g++' I guess. You need to do something similar to what is
> done in the 'maradns' package.
I never thought of testing that, I should start doing it. Fixed in v2.
[0]: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=40607#28
- Brice
^ permalink raw reply [flat|nested] 5+ messages in thread
* [bug#40902] [PATCH v2] gnu: Add edid-decode.
2020-04-27 13:55 [bug#40902] [PATCH] gnu: Add edid-decode Brice Waegeneire
2020-04-28 7:46 ` Mathieu Othacehe
@ 2020-04-28 9:53 ` Brice Waegeneire
2020-04-29 9:41 ` bug#40902: " Mathieu Othacehe
1 sibling, 1 reply; 5+ messages in thread
From: Brice Waegeneire @ 2020-04-28 9:53 UTC (permalink / raw)
To: 40902; +Cc: m.othacehe
* gnu/packages/hardware.scm (edid-decode): New variable.
---
Fix identation. Add phase 'fix-cross-compilation'.
gnu/packages/hardware.scm | 37 +++++++++++++++++++++++++++++++++++++
1 file changed, 37 insertions(+)
diff --git a/gnu/packages/hardware.scm b/gnu/packages/hardware.scm
index 189c73ee3c..0128832f28 100644
--- a/gnu/packages/hardware.scm
+++ b/gnu/packages/hardware.scm
@@ -89,6 +89,43 @@ calibrated, and restored when the calibration is applied.")
(license (list license:bsd-3 ; FindDDCUtil.cmake
license:gpl2+)))) ; everything else
+(define-public edid-decode
+ (let ((commit "74b64180d67bb009d8d9ea1b6f18ad41aaa16396") ; 2020-04-22
+ (revision "1"))
+ (package
+ (name "edid-decode")
+ (version (git-version "0.0.0" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (file-name (git-file-name name version))
+ (uri (git-reference
+ (url "git://linuxtv.org/edid-decode.git")
+ (commit commit)))
+ (sha256
+ (base32 "0nirp5bza08zj5d8bjgcm0p869hdg3qg3mwa7999pjdrzmn7s2ah"))))
+ (build-system gnu-build-system)
+ (arguments
+ `(#:tests? #f ; No test suite
+ #:make-flags
+ (list (string-append "DESTDIR=" (assoc-ref %outputs "out"))
+ "bindir=/bin" "mandir=/share/man")
+ #:phases
+ (modify-phases %standard-phases
+ (add-after 'unpack 'fix-cross-compilation
+ (lambda* (#:key native-inputs target #:allow-other-keys)
+ (when target
+ (substitute* "Makefile"
+ (("\\$\\(CXX\\)")
+ (string-append (assoc-ref native-inputs "gcc") "/bin/g++"))))
+ #t))
+ (delete 'configure))))
+ (home-page "https://git.linuxtv.org/edid-decode.git/")
+ (synopsis "Decode @dfn{EDID} data in human-readable format")
+ (description "edid-decode decodes @dfn{EDID} monitor description data in
+human-readable format and check if it conforms to the standards.")
+ (license license:expat))))
+
;; Distinct from memtest86, which is obsolete.
(define-public memtest86+
(package
--
2.26.0
^ permalink raw reply related [flat|nested] 5+ messages in thread
* bug#40902: [PATCH v2] gnu: Add edid-decode.
2020-04-28 9:53 ` [bug#40902] [PATCH v2] " Brice Waegeneire
@ 2020-04-29 9:41 ` Mathieu Othacehe
0 siblings, 0 replies; 5+ messages in thread
From: Mathieu Othacehe @ 2020-04-29 9:41 UTC (permalink / raw)
To: Brice Waegeneire; +Cc: 40902-done
Hello Brice,
> + (when target
> + (substitute* "Makefile"
> + (("\\$\\(CXX\\)")
> + (string-append (assoc-ref native-inputs "gcc") "/bin/g++"))))
Doing that you are still using the native compiler. I changed it to:
--8<---------------cut here---------------start------------->8---
(when target
(substitute* "Makefile"
(("\\$\\(CXX\\)")
(string-append target "-g++"))))
--8<---------------cut here---------------end--------------->8---
and pushed!
Thanks,
Mathieu
^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2020-04-29 9:42 UTC | newest]
Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-04-27 13:55 [bug#40902] [PATCH] gnu: Add edid-decode Brice Waegeneire
2020-04-28 7:46 ` Mathieu Othacehe
2020-04-28 9:54 ` Brice Waegeneire
2020-04-28 9:53 ` [bug#40902] [PATCH v2] " Brice Waegeneire
2020-04-29 9:41 ` bug#40902: " Mathieu Othacehe
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).