unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: wrap-python3: create more symlinks.
  2016-04-21 20:23 Hartmut Goebel
@ 2016-04-21 20:23 ` Hartmut Goebel
  2016-04-30  2:05   ` Leo Famulari
  0 siblings, 1 reply; 17+ messages in thread
From: Hartmut Goebel @ 2016-04-21 20:23 UTC (permalink / raw)
  To: guix-devel

* gnu/pyckages/python.csm (wrap-python3): Create symlinks for
  pip and python-config, too.
---
 gnu/packages/python.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 65210e0..5c4bf3b 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -18,6 +18,7 @@
 ;;; Copyright © 2015 Kyle Meyer <kyle@kyleam.com>
 ;;; Copyright © 2015, 2016 Chris Marusich <cmmarusich@gmail.com>
 ;;; Copyright © 2016 Danny Milosavljevic <dannym+a@scratchpost.org>
+;;; Copyright © 2016 Hartmut Goebel <h.goebel@crazy-compilers.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -334,8 +335,8 @@ data types.")
                   (lambda (old new)
                     (symlink (string-append python old)
                              (string-append bin "/" new)))
-                  `("python3" ,"pydoc3" ,"idle3")
-                  `("python"  ,"pydoc"  ,"idle"))))))
+                  `("python3" ,"pydoc3" ,"idle3" ,"pip3" ,"python3-config")
+                  `("python"  ,"pydoc"  ,"idle"  ,"pip"  ,"python-config"))))))
     (synopsis "Wrapper for the Python 3 commands")
     (description
      "This package provides wrappers for the commands of Python@tie{}3.x such
-- 
2.7.4

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-04-21 20:23 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
@ 2016-04-30  2:05   ` Leo Famulari
  2016-05-05 14:58     ` Ludovic Courtès
  0 siblings, 1 reply; 17+ messages in thread
From: Leo Famulari @ 2016-04-30  2:05 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

On Thu, Apr 21, 2016 at 10:23:19PM +0200, Hartmut Goebel wrote:
> * gnu/pyckages/python.csm (wrap-python3): Create symlinks for
>   pip and python-config, too.

[...]

> +++ b/gnu/packages/python.scm
> -                  `("python3" ,"pydoc3" ,"idle3")
> -                  `("python"  ,"pydoc"  ,"idle"))))))
> +                  `("python3" ,"pydoc3" ,"idle3" ,"pip3" ,"python3-config")
> +                  `("python"  ,"pydoc"  ,"idle"  ,"pip"  ,"python-config"))))))

Do any Pythonista's have any comments on this proposal?

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-04-30  2:05   ` Leo Famulari
@ 2016-05-05 14:58     ` Ludovic Courtès
  2016-05-05 16:41       ` Alex Kost
  0 siblings, 1 reply; 17+ messages in thread
From: Ludovic Courtès @ 2016-05-05 14:58 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari <leo@famulari.name> skribis:

> On Thu, Apr 21, 2016 at 10:23:19PM +0200, Hartmut Goebel wrote:
>> * gnu/pyckages/python.csm (wrap-python3): Create symlinks for
>>   pip and python-config, too.
>
> [...]
>
>> +++ b/gnu/packages/python.scm
>> -                  `("python3" ,"pydoc3" ,"idle3")
>> -                  `("python"  ,"pydoc"  ,"idle"))))))
>> +                  `("python3" ,"pydoc3" ,"idle3" ,"pip3" ,"python3-config")
>> +                  `("python"  ,"pydoc"  ,"idle"  ,"pip"  ,"python-config"))))))
>
> Do any Pythonista's have any comments on this proposal?

I’m not much of a Pythonista, but this look good to me.
Could you push to core-updates?

Thanks!

Ludo’.

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-05-05 14:58     ` Ludovic Courtès
@ 2016-05-05 16:41       ` Alex Kost
  2016-05-06 18:15         ` Leo Famulari
  0 siblings, 1 reply; 17+ messages in thread
From: Alex Kost @ 2016-05-05 16:41 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Ludovic Courtès (2016-05-05 17:58 +0300) wrote:

> Leo Famulari <leo@famulari.name> skribis:
>
>> On Thu, Apr 21, 2016 at 10:23:19PM +0200, Hartmut Goebel wrote:
>>> * gnu/pyckages/python.csm (wrap-python3): Create symlinks for
>>>   pip and python-config, too.
>>
>> [...]
>>
>>> +++ b/gnu/packages/python.scm
>>> -                  `("python3" ,"pydoc3" ,"idle3")
>>> -                  `("python"  ,"pydoc"  ,"idle"))))))
>>> +                  `("python3" ,"pydoc3" ,"idle3" ,"pip3" ,"python3-config")
>>> +                  `("python"  ,"pydoc"  ,"idle"  ,"pip"  ,"python-config"))))))
>>
>> Do any Pythonista's have any comments on this proposal?
>
> I’m not much of a Pythonista, but this look good to me.
> Could you push to core-updates?

I was going to say that these lists of strings are strangely quoted, but
then I saw that Mark fixed it in the core-updates¹, so this patch should
be adjusted accordingly.

¹ http://git.savannah.gnu.org/cgit/guix.git/commit/?id=5c19dab94591db68c5361bc2521321fc587dc704

-- 
Alex

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-05-05 16:41       ` Alex Kost
@ 2016-05-06 18:15         ` Leo Famulari
  0 siblings, 0 replies; 17+ messages in thread
From: Leo Famulari @ 2016-05-06 18:15 UTC (permalink / raw)
  To: Alex Kost; +Cc: guix-devel

On Thu, May 05, 2016 at 07:41:02PM +0300, Alex Kost wrote:
> Ludovic Courtès (2016-05-05 17:58 +0300) wrote:
> 
> > Leo Famulari <leo@famulari.name> skribis:
> >
> >> On Thu, Apr 21, 2016 at 10:23:19PM +0200, Hartmut Goebel wrote:
> >>> * gnu/pyckages/python.csm (wrap-python3): Create symlinks for
> >>>   pip and python-config, too.
> >>
> >> [...]
> >>
> >>> +++ b/gnu/packages/python.scm
> >>> -                  `("python3" ,"pydoc3" ,"idle3")
> >>> -                  `("python"  ,"pydoc"  ,"idle"))))))
> >>> +                  `("python3" ,"pydoc3" ,"idle3" ,"pip3" ,"python3-config")
> >>> +                  `("python"  ,"pydoc"  ,"idle"  ,"pip"  ,"python-config"))))))
> >>
> >> Do any Pythonista's have any comments on this proposal?
> >
> > I’m not much of a Pythonista, but this look good to me.
> > Could you push to core-updates?
> 
> I was going to say that these lists of strings are strangely quoted, but
> then I saw that Mark fixed it in the core-updates¹, so this patch should
> be adjusted accordingly.

Committed to core-updates with the requested changes as 478e75b53b.

> 
> ¹ http://git.savannah.gnu.org/cgit/guix.git/commit/?id=5c19dab94591db68c5361bc2521321fc587dc704
> 
> -- 
> Alex

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

* [PATCH] Add more symlinks to python-wrapper
@ 2016-06-14 21:14 Hartmut Goebel
  2016-06-14 21:14 ` [PATCH v5] gnu: Add teensy-loader-cli Hartmut Goebel
                   ` (2 more replies)
  0 siblings, 3 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-14 21:14 UTC (permalink / raw)
  To: guix-devel

Enclosed patch adds missing sysmlinks "python-config" and "pip" to
python-wrapper.

I did not add a symlink for easy_install, this this is obsoleted by pip.

Hartmut Goebel (1):
  gnu: wrap-python3: create more symlinks.

 gnu/packages/python.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

-- 
2.7.4

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

* [PATCH v5] gnu: Add teensy-loader-cli.
  2016-06-14 21:14 [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
@ 2016-06-14 21:14 ` Hartmut Goebel
  2016-06-14 21:16   ` Hartmut Goebel
  2016-06-15 13:31   ` Ludovic Courtès
  2016-06-14 21:14 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
  2016-06-14 21:16 ` [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
  2 siblings, 2 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-14 21:14 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/flashing-tools.scm (teensy-loader-cli): New variable.
* gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch,
  gnu/packages/patches/teensy-loader-cli-Change-program-name-in-usage.patch
  New files.
---
 gnu/packages/flashing-tools.scm                    | 53 ++++++++++++++++++++++
 .../teensy-loader-cli-Add-support-for-h-help.patch | 34 ++++++++++++++
 ...y-loader-cli-Change-program-name-in-usage.patch | 33 ++++++++++++++
 3 files changed, 120 insertions(+)
 create mode 100644 gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch
 create mode 100644 gnu/packages/patches/teensy-loader-cli-Change-program-name-in-usage.patch

diff --git a/gnu/packages/flashing-tools.scm b/gnu/packages/flashing-tools.scm
index 37a1072..65f68a6 100644
--- a/gnu/packages/flashing-tools.scm
+++ b/gnu/packages/flashing-tools.scm
@@ -160,3 +160,56 @@ With dfu-util you are able to download firmware to your device or upload
 firmware from it.")
     (home-page "http://dfu-util.sourceforge.net/")
     (license gpl2+)))
+
+(define-public teensy-loader-cli
+  ;; The repo does not tag versions nor does it use releases, but a commit
+  ;; message says "Importing 2.1", while the sourcce still says "2.0". So pin
+  ;; to a fixed commit.
+  (let ((commit "f289b7a2e5627464044249f0e5742830e052e360"))
+    (package
+      (name "teensy-loader-cli")
+      (version (string-append "2.1-1." (string-take commit 7)))
+      (source
+       (origin
+         (method url-fetch)
+         (uri (string-append "https://github.com/PaulStoffregen/"
+                             "teensy_loader_cli/archive/" commit ".tar.gz"))
+         (sha256 (base32 "17wqc2q4fa473cy7f5m2yiyb9nq0qw7xal2kzrxzaikgm9rabsw8"))
+         (file-name (string-append "teensy-loader-cli-" version ".tar.gz" ))
+         (modules '((guix build utils)))
+         (snippet
+          `(begin
+             ;; Remove example flash files and teensy rebooter flash binaries.
+             (for-each delete-file (find-files "." "\\.(elf|hex)$"))
+             ;; Fix the version
+             (substitute* "teensy_loader_cli.c"
+               (("Teensy Loader, Command Line, Version 2.0\\\\n")
+                (string-append "Teensy Loader, Command Line, " ,version "\\n")))
+             #t))
+       (patches (search-patches
+                 "teensy-loader-cli-Add-support-for-h-help.patch"
+                 "teensy-loader-cli-Change-program-name-in-usage.patch"))))
+      (build-system gnu-build-system)
+      (arguments
+       '(#:tests? #f ;; Makefile has no test target
+         #:make-flags (list "CC=gcc" (string-append "PREFIX=" %output))
+         #:phases
+         (modify-phases %standard-phases
+           (delete 'configure)
+           (replace 'install
+             (lambda* (#:key outputs #:allow-other-keys)
+               (let* ((out (assoc-ref outputs "out"))
+                      (bin (string-append out "/bin")))
+                 (rename-file "teensy_loader_cli" "teensy-loader-cli")
+                 (install-file "teensy-loader-cli" bin)))))))
+      (inputs
+       `(("libusb-compat" ,libusb-compat)))
+      (synopsis "Command line firmware uploader for Teensy development boards")
+      (description "The Teensy Loader program communicates with your
+Teensy board when the HalfKay bootloader is running, so you can
+download new programs and run them.
+
+You'll need to add the udev-rules to make the teensy update available
+for non-root users.")
+      (home-page "https://www.pjrc.com/teensy/loader_cli.html")
+      (license gpl3))))
diff --git a/gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch b/gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch
new file mode 100644
index 0000000..a5e0128
--- /dev/null
+++ b/gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch
@@ -0,0 +1,34 @@
+Add support for `-h` (help).
+This option was already in the usage message, but not implemented.
+
+Upstream pull-request pending.
+
+---
+ teensy_loader_cli.c | 4 +++-
+ 1 file changed, 3 insertions(+), 1 deletion(-)
+
+diff --git a/teensy_loader_cli.c b/teensy_loader_cli.c
+index d4a6cc5..0a06209 100644
+--- a/teensy_loader_cli.c
++++ b/teensy_loader_cli.c
+@@ -37,7 +37,8 @@ void usage(const char *err)
+ {
+ 	if(err != NULL) fprintf(stderr, "%s\n\n", err);
+ 	fprintf(stderr,
+-		"Usage: teensy_loader_cli --mcu=<MCU> [-w] [-h] [-n] [-b] [-v] <file.hex>\n"
++		"Usage: teensy_loader_cli --mcu=<MCU> [-h] [-w] [-n] [-b] [-v] <file.hex>\n"
++		"\t-h : Print this help message\n"
+ 		"\t-w : Wait for device to appear\n"
+ 		"\t-r : Use hard reboot if device not online\n"
+ 		"\t-s : Use soft reboot if device not online (Teensy3.x only)\n"
+@@ -1081,6 +1082,7 @@ void parse_flag(char *arg)
+ 	int i;
+ 	for(i=1; arg[i]; i++) {
+ 		switch(arg[i]) {
++			case 'h': usage(NULL); break;
+ 			case 'w': wait_for_device_to_appear = 1; break;
+ 			case 'r': hard_reboot_device = 1; break;
+ 			case 's': soft_reboot_device = 1; break;
+-- 
+2.7.4
+
diff --git a/gnu/packages/patches/teensy-loader-cli-Change-program-name-in-usage.patch b/gnu/packages/patches/teensy-loader-cli-Change-program-name-in-usage.patch
new file mode 100644
index 0000000..8773237
--- /dev/null
+++ b/gnu/packages/patches/teensy-loader-cli-Change-program-name-in-usage.patch
@@ -0,0 +1,33 @@
+Change program name in usage: use dashes instead of underscore.
+
+Not pushed upstream since the official name is using underscores.
+
+---
+ teensy_loader_cli.c | 4 ++--
+ 1 file changed, 2 insertions(+), 2 deletions(-)
+
+diff --git a/teensy_loader_cli.c b/teensy_loader_cli.c
+index 0a06209..049954e 100644
+--- a/teensy_loader_cli.c
++++ b/teensy_loader_cli.c
+@@ -37,7 +37,7 @@ void usage(const char *err)
+ {
+ 	if(err != NULL) fprintf(stderr, "%s\n\n", err);
+ 	fprintf(stderr,
+-		"Usage: teensy_loader_cli --mcu=<MCU> [-h] [-w] [-n] [-b] [-v] <file.hex>\n"
++		"Usage: teensy-loader-cli --mcu=<MCU> [-h] [-w] [-n] [-b] [-v] <file.hex>\n"
+ 		"\t-h : Print this help message\n"
+ 		"\t-w : Wait for device to appear\n"
+ 		"\t-r : Use hard reboot if device not online\n"
+@@ -45,7 +45,7 @@ void usage(const char *err)
+ 		"\t-n : No reboot after programming\n"
+ 		"\t-b : Boot only, do not program\n"
+ 		"\t-v : Verbose output\n"
+-		"\nUse `teensy_loader_cli --list-mcus` to list supported MCUs.\n"
++		"\nUse `teensy-loader-cli --list-mcus` to list supported MCUs.\n"
+ 		"\nFor more information, please visit:\n"
+ 		"http://www.pjrc.com/teensy/loader_cli.html\n");
+ 	exit(1);
+-- 
+2.7.4
+
-- 
2.7.4

Update v5:
* Use quasiquote and eliminate "version" variable.
* Fix spelling and typing.
* Fix indention.
* Split patch into two, add explanation and upstream status.
Update v4:
* use a "snippet" to remove binary files and fix version
Update v3:
* Use long commit hash
* remove binary files
Update: v2:
* Include patch file

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

* [PATCH] gnu: wrap-python3: create more symlinks.
  2016-06-14 21:14 [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
  2016-06-14 21:14 ` [PATCH v5] gnu: Add teensy-loader-cli Hartmut Goebel
@ 2016-06-14 21:14 ` Hartmut Goebel
  2016-06-14 21:17   ` Hartmut Goebel
  2016-06-15 14:27   ` Andreas Enge
  2016-06-14 21:16 ` [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
  2 siblings, 2 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-14 21:14 UTC (permalink / raw)
  To: guix-devel

* gnu/pyckages/python.csm (wrap-python3): Create symlinks for
  pip and python-config, too.
---
 gnu/packages/python.scm | 5 +++--
 1 file changed, 3 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 65210e0..5c4bf3b 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -18,6 +18,7 @@
 ;;; Copyright © 2015 Kyle Meyer <kyle@kyleam.com>
 ;;; Copyright © 2015, 2016 Chris Marusich <cmmarusich@gmail.com>
 ;;; Copyright © 2016 Danny Milosavljevic <dannym+a@scratchpost.org>
+;;; Copyright © 2016 Hartmut Goebel <h.goebel@crazy-compilers.com>
 ;;;
 ;;; This file is part of GNU Guix.
 ;;;
@@ -334,8 +335,8 @@ data types.")
                   (lambda (old new)
                     (symlink (string-append python old)
                              (string-append bin "/" new)))
-                  `("python3" ,"pydoc3" ,"idle3")
-                  `("python"  ,"pydoc"  ,"idle"))))))
+                  `("python3" ,"pydoc3" ,"idle3" ,"pip3" ,"python3-config")
+                  `("python"  ,"pydoc"  ,"idle"  ,"pip"  ,"python-config"))))))
     (synopsis "Wrapper for the Python 3 commands")
     (description
      "This package provides wrappers for the commands of Python@tie{}3.x such
-- 
2.7.4

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

* Re: [PATCH] Add more symlinks to python-wrapper
  2016-06-14 21:14 [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
  2016-06-14 21:14 ` [PATCH v5] gnu: Add teensy-loader-cli Hartmut Goebel
  2016-06-14 21:14 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
@ 2016-06-14 21:16 ` Hartmut Goebel
  2 siblings, 0 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-14 21:16 UTC (permalink / raw)
  To: guix-devel


[-- Attachment #1.1: Type: text/plain, Size: 469 bytes --]

Sorry, this patch was accidentaly resend.

-- 
Schönen Gruß
Hartmut Goebel
Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer
Information Security Management, Security Governance, Secure Software
Development

Goebel Consult, Landshut
http://www.goebel-consult.de

Blog:
http://www.goebel-consult.de/blog/liberario-als-alternative-zum-db-navigator-und-zu-offi

Kolumne: http://www.cissp-gefluester.de/2010-01-hinterturen-allen-ortes


[-- Attachment #1.2: Type: text/html, Size: 1425 bytes --]

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 2430 bytes --]

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

* Re: [PATCH v5] gnu: Add teensy-loader-cli.
  2016-06-14 21:14 ` [PATCH v5] gnu: Add teensy-loader-cli Hartmut Goebel
@ 2016-06-14 21:16   ` Hartmut Goebel
  2016-06-15 13:31   ` Ludovic Courtès
  1 sibling, 0 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-14 21:16 UTC (permalink / raw)
  To: guix-devel

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

Sorry, this patch was accidentaly resend.


[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 2430 bytes --]

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-06-14 21:14 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
@ 2016-06-14 21:17   ` Hartmut Goebel
  2016-06-15 14:27   ` Andreas Enge
  1 sibling, 0 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-14 21:17 UTC (permalink / raw)
  To: guix-devel


[-- Attachment #1.1: Type: text/plain, Size: 469 bytes --]

Sorry, this patch was accidentaly resend.

-- 
Schönen Gruß
Hartmut Goebel
Dipl.-Informatiker (univ), CISSP, CSSLP, ISO 27001 Lead Implementer
Information Security Management, Security Governance, Secure Software
Development

Goebel Consult, Landshut
http://www.goebel-consult.de

Blog:
http://www.goebel-consult.de/blog/liberario-als-alternative-zum-db-navigator-und-zu-offi

Kolumne: http://www.cissp-gefluester.de/2010-01-hinterturen-allen-ortes


[-- Attachment #1.2: Type: text/html, Size: 1425 bytes --]

[-- Attachment #2: S/MIME Cryptographic Signature --]
[-- Type: application/pkcs7-signature, Size: 2430 bytes --]

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

* Re: [PATCH v5] gnu: Add teensy-loader-cli.
  2016-06-14 21:14 ` [PATCH v5] gnu: Add teensy-loader-cli Hartmut Goebel
  2016-06-14 21:16   ` Hartmut Goebel
@ 2016-06-15 13:31   ` Ludovic Courtès
  2016-06-15 13:34     ` Hartmut Goebel
  1 sibling, 1 reply; 17+ messages in thread
From: Ludovic Courtès @ 2016-06-15 13:31 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

Hello,

Hartmut Goebel <h.goebel@crazy-compilers.com> skribis:

> * gnu/packages/flashing-tools.scm (teensy-loader-cli): New variable.
> * gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch,
>   gnu/packages/patches/teensy-loader-cli-Change-program-name-in-usage.patch
>   New files.

I’ve finally applied it and took the liberty to make a few changes (I
didn’t want to bother you another time!).

> +       (patches (search-patches
> +                 "teensy-loader-cli-Add-support-for-h-help.patch"
> +                 "teensy-loader-cli-Change-program-name-in-usage.patch"))))

I removed the second patch and corresponding renaming (I hadn’t noticed
it before), on the grounds that it’s not our job to rename things; in
general, we should stick to what upstream has chosen.

I’ve also added the remaining patch to gnu/local.mk.

> +++ b/gnu/packages/patches/teensy-loader-cli-Add-support-for-h-help.patch
> @@ -0,0 +1,34 @@
> +Add support for `-h` (help).
> +This option was already in the usage message, but not implemented.
> +
> +Upstream pull-request pending.

I added the URL of the pull request instead.

Note that, normally, we would not incorporate such patches, because it’s
really upstream’s business, not ours.

Thank you!

Ludo’.

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

* Re: [PATCH v5] gnu: Add teensy-loader-cli.
  2016-06-15 13:31   ` Ludovic Courtès
@ 2016-06-15 13:34     ` Hartmut Goebel
  0 siblings, 0 replies; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-15 13:34 UTC (permalink / raw)
  To: Ludovic Courtès; +Cc: guix-devel

Am 15.06.2016 um 15:31 schrieb Ludovic Courtès:
> I’ve finally applied it and took the liberty to make a few changes (I
> didn’t want to bother you another time!).

Thanks :-)

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-06-14 21:14 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
  2016-06-14 21:17   ` Hartmut Goebel
@ 2016-06-15 14:27   ` Andreas Enge
  2016-06-17 19:45     ` Hartmut Goebel
  1 sibling, 1 reply; 17+ messages in thread
From: Andreas Enge @ 2016-06-15 14:27 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

Hi Hartmut,

the content of the patch looks good, but I cannot apply it with "git am";
is it relative to an older git commit? I could correct it myself, but
then you would disappear as an author.

Andreas

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-06-15 14:27   ` Andreas Enge
@ 2016-06-17 19:45     ` Hartmut Goebel
  2016-06-20  9:10       ` Andreas Enge
  0 siblings, 1 reply; 17+ messages in thread
From: Hartmut Goebel @ 2016-06-17 19:45 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

Hello Andreas,

Am 15.06.2016 um 16:27 schrieb Andreas Enge:
> the content of the patch looks good, but I cannot apply it with "git am";
> is it relative to an older git commit? I could correct it myself, but
> then you would disappear as an author.

I've jsut send an updated version of the patch. I'd appreciate if you
could take care of it.

-- 
Regards
Hartmut Goebel

| Hartmut Goebel          | h.goebel@crazy-compilers.com               |
| www.crazy-compilers.com | compilers which you thought are impossible |

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-06-17 19:45     ` Hartmut Goebel
@ 2016-06-20  9:10       ` Andreas Enge
  2016-06-23 19:57         ` Andreas Enge
  0 siblings, 1 reply; 17+ messages in thread
From: Andreas Enge @ 2016-06-20  9:10 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

Hello Hartmut,

On Fri, Jun 17, 2016 at 09:45:22PM +0200, Hartmut Goebel wrote:
> I've jsut send an updated version of the patch. I'd appreciate if you
> could take care of it.

thanks a lot! I applied it locally and was about to push it to master,
when I started to wonder:
$ ./pre-inst-env guix refresh -l python-wrapper
Building the following 558 packages would ensure 1569 dependent packages are rebuilt

This is maybe too much right now; question to those who suffer the most from
core-updates: Should I push it to core-updates-next? This patch is nice to
have, but probably not very urgent.

Andreas

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

* Re: [PATCH] gnu: wrap-python3: create more symlinks.
  2016-06-20  9:10       ` Andreas Enge
@ 2016-06-23 19:57         ` Andreas Enge
  0 siblings, 0 replies; 17+ messages in thread
From: Andreas Enge @ 2016-06-23 19:57 UTC (permalink / raw)
  To: Hartmut Goebel; +Cc: guix-devel

On Mon, Jun 20, 2016 at 11:10:20AM +0200, Andreas Enge wrote:
> This is maybe too much right now; question to those who suffer the most from
> core-updates: Should I push it to core-updates-next? This patch is nice to
> have, but probably not very urgent.

I just pushed to core-updates-next.

Andreas

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

end of thread, other threads:[~2016-06-23 19:57 UTC | newest]

Thread overview: 17+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-06-14 21:14 [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
2016-06-14 21:14 ` [PATCH v5] gnu: Add teensy-loader-cli Hartmut Goebel
2016-06-14 21:16   ` Hartmut Goebel
2016-06-15 13:31   ` Ludovic Courtès
2016-06-15 13:34     ` Hartmut Goebel
2016-06-14 21:14 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
2016-06-14 21:17   ` Hartmut Goebel
2016-06-15 14:27   ` Andreas Enge
2016-06-17 19:45     ` Hartmut Goebel
2016-06-20  9:10       ` Andreas Enge
2016-06-23 19:57         ` Andreas Enge
2016-06-14 21:16 ` [PATCH] Add more symlinks to python-wrapper Hartmut Goebel
  -- strict thread matches above, loose matches on Subject: below --
2016-04-21 20:23 Hartmut Goebel
2016-04-21 20:23 ` [PATCH] gnu: wrap-python3: create more symlinks Hartmut Goebel
2016-04-30  2:05   ` Leo Famulari
2016-05-05 14:58     ` Ludovic Courtès
2016-05-05 16:41       ` Alex Kost
2016-05-06 18:15         ` Leo Famulari

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