unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH 0/4] Add more dependencies of OpenStack.
@ 2015-09-14 21:23 Cyril Roelandt
  2015-09-14 21:23 ` [PATCH 1/4] gnu: Add python-oslo.serialization Cyril Roelandt
                   ` (4 more replies)
  0 siblings, 5 replies; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-14 21:23 UTC (permalink / raw)
  To: guix-devel

These packages would be really useful to have for a demo of guix-tox :)

--
Cyril Roelandt (4):
  gnu: Add python-oslo.serialization.
  gnu: Add python-oslo.context.
  gnu: Add python-webob.
  gnu: Add python-requests-mock.

 gnu/packages/openstack.scm | 101 +++++++++++++++++++++++++++++++++++++++++++++
 gnu/packages/python.scm    |  27 ++++++++++++
 2 files changed, 128 insertions(+)

-- 
2.1.4

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

* [PATCH 1/4] gnu: Add python-oslo.serialization.
  2015-09-14 21:23 [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
@ 2015-09-14 21:23 ` Cyril Roelandt
  2015-09-15 16:34   ` Mathieu Lirzin
  2015-09-14 21:23 ` [PATCH 2/4] gnu: Add python-oslo.context Cyril Roelandt
                   ` (3 subsequent siblings)
  4 siblings, 1 reply; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-14 21:23 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/openstack.scm (python-oslo.serialization,
  python2-oslo.serialization): New variables.
---
 gnu/packages/openstack.scm | 39 +++++++++++++++++++++++++++++++++++++++
 1 file changed, 39 insertions(+)

diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm
index a9db3f2..04ddbb3 100644
--- a/gnu/packages/openstack.scm
+++ b/gnu/packages/openstack.scm
@@ -290,6 +290,45 @@ in an application or library.")
 (define-public python2-oslo.i18n
   (package-with-python2 python-oslo.i18n))
 
+(define-public python-oslo.serialization
+  (package
+    (name "python-oslo.serialization")
+    (version "1.9.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://pypi.python.org/packages/source/o/oslo.serialization/"
+             "oslo.serialization-" version ".tar.gz"))
+       (sha256
+         (base32
+          "00qaxg155s61ylh4fqc7m5fh0gijf33khhai9xvcsc9k106i3c9c"))))
+    (build-system python-build-system)
+    (propagated-inputs
+      `(("python-iso8601" ,python-iso8601)
+        ("python-netaddr" ,python-netaddr)
+        ("python-oslo.utils" ,python-oslo.utils)
+        ("python-simplejson" ,python-simplejson)
+        ("python-six" ,python-six)
+        ("python-pytz" ,python-pytz)))
+    (inputs
+      `(("python-babel" ,python-babel)
+        ("python-pbr" ,python-pbr)
+        ("python-setuptools" ,python-setuptools)
+        ;; Tests.
+        ("python-mock" ,python-mock)
+        ("python-oslo.i18n" ,python-oslo.i18n)
+        ("python-oslotest" ,python-oslotest)))
+    (home-page "http://launchpad.net/oslo")
+    (synopsis "Oslo serialization library")
+    (description
+      "The oslo.serialization library provides support for representing objects
+in transmittable and storable formats, such as JSON and MessagePack.")
+    (license asl2.0)))
+
+(define-public python2-oslo.serialization
+  (package-with-python2 python-oslo.serialization))
+
 (define-public python-oslotest
   (package
     (name "python-oslotest")
-- 
2.1.4

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

* [PATCH 2/4] gnu: Add python-oslo.context.
  2015-09-14 21:23 [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
  2015-09-14 21:23 ` [PATCH 1/4] gnu: Add python-oslo.serialization Cyril Roelandt
@ 2015-09-14 21:23 ` Cyril Roelandt
  2015-09-15 16:36   ` Mathieu Lirzin
  2015-09-14 21:23 ` [PATCH 3/4] gnu: Add python-webob Cyril Roelandt
                   ` (2 subsequent siblings)
  4 siblings, 1 reply; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-14 21:23 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/openstack.scm (python-oslo.context,
  python2-oslo.context): New variables.
---
 gnu/packages/openstack.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm
index 04ddbb3..1a2af79 100644
--- a/gnu/packages/openstack.scm
+++ b/gnu/packages/openstack.scm
@@ -253,6 +253,37 @@ extensions.")
 (define-public python2-oslo.config
   (package-with-python2 python-oslo.config))
 
+(define-public python-oslo.context
+  (package
+    (name "python-oslo.context")
+    (version "0.6.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://pypi.python.org/packages/source/o/oslo.context/"
+             "oslo.context-" version ".tar.gz"))
+       (sha256
+         (base32
+          "16wr9qrkc3lb94ssb14qid4liza66x316fvzjw0izg67h1a0fm86"))))
+    (build-system python-build-system)
+    (inputs
+      `(("python-babel" ,python-babel)
+        ("python-pbr" ,python-pbr)
+        ("python-setuptools" ,python-setuptools)
+        ;; Tests.
+        ("python-oslotest" ,python-oslotest)))
+    (home-page "http://launchpad.net/oslo")
+    (synopsis "Oslo context library")
+    (description
+      "The Oslo context library has helpers to maintain useful information
+about a request context. The request context is usually populated in the WSGI
+pipeline and used by various modules such as logging.")
+    (license asl2.0)))
+
+(define-public python2-oslo.context
+  (package-with-python2 python-oslo.context))
+
 (define-public python-oslo.i18n
   (package
     (name "python-oslo.i18n")
-- 
2.1.4

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

* [PATCH 3/4] gnu: Add python-webob.
  2015-09-14 21:23 [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
  2015-09-14 21:23 ` [PATCH 1/4] gnu: Add python-oslo.serialization Cyril Roelandt
  2015-09-14 21:23 ` [PATCH 2/4] gnu: Add python-oslo.context Cyril Roelandt
@ 2015-09-14 21:23 ` Cyril Roelandt
  2015-09-15 16:43   ` Mathieu Lirzin
  2015-09-14 21:23 ` [PATCH 4/4] gnu: Add python-requests-mock Cyril Roelandt
  2015-09-16 20:54 ` [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
  4 siblings, 1 reply; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-14 21:23 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/python.scm (python-webob, python2-webob): New variables.
---
 gnu/packages/python.scm | 27 +++++++++++++++++++++++++++
 1 file changed, 27 insertions(+)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index c58d7f3..919573f 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -4853,3 +4853,30 @@ fractional seconds) of a clock which never goes backwards.")
 
 (define-public python2-monotonic
   (package-with-python2 python-monotonic))
+
+(define-public python-webob
+  (package
+    (name "python-WebOb")
+    (version "1.5.0b0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://pypi.python.org/packages/source/W/WebOb/WebOb-"
+             version ".tar.gz"))
+       (sha256
+         (base32
+          "140b3iczclk1j0405rvw5gxshqfkhcc8254fj520z3m23cwbql4a"))))
+    (build-system python-build-system)
+    (inputs
+      `(("python-nose" ,python-nose)
+        ("python-setuptools" ,python-setuptools)))
+    (home-page "http://webob.org/")
+    (synopsis "WSGI request and response object")
+    (description
+      "WebOb provides wrappers around the WSGI request environment, and an
+object to help create WSGI responses.")
+    (license license:expat)))
+
+(define-public python2-webob
+  (package-with-python2 python-webob))
-- 
2.1.4

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

* [PATCH 4/4] gnu: Add python-requests-mock.
  2015-09-14 21:23 [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
                   ` (2 preceding siblings ...)
  2015-09-14 21:23 ` [PATCH 3/4] gnu: Add python-webob Cyril Roelandt
@ 2015-09-14 21:23 ` Cyril Roelandt
  2015-09-15 16:54   ` Mathieu Lirzin
  2015-09-16 20:54 ` [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
  4 siblings, 1 reply; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-14 21:23 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/openstack.scm (python-requests-mock,
  python2-requests-mock): New variables.
---
 gnu/packages/openstack.scm | 31 +++++++++++++++++++++++++++++++
 1 file changed, 31 insertions(+)

diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm
index 1a2af79..2f3e883 100644
--- a/gnu/packages/openstack.scm
+++ b/gnu/packages/openstack.scm
@@ -174,6 +174,37 @@ and sensible default behaviors into your setuptools run.")
 (define-public python2-pbr
   (package-with-python2 python-pbr))
 
+(define-public python-requests-mock
+  (package
+    (name "python-requests-mock")
+    (version "0.6.0")
+    (source
+     (origin
+       (method url-fetch)
+       (uri (string-append
+             "https://pypi.python.org/packages/source/r/requests-mock/"
+             "requests-mock-" version ".tar.gz"))
+       (sha256
+         (base32
+          "0gmd88c224y53b1ai8cfsrcxm9kw3gdqzysclmnaqspg7zjhxwd1"))))
+    (build-system python-build-system)
+    (propagated-inputs
+      `(("python-requests" ,python-requests)
+        ("python-six" ,python-six)))
+    (inputs
+      `(("python-mock" ,python-mock)
+        ("python-pbr" ,python-pbr)
+        ("python-setuptools" ,python-setuptools)))
+    (home-page "https://requests-mock.readthedocs.org/")
+    (synopsis "Mock out responses from the requests package")
+    (description
+      "This module provides a building block to stub out the HTTP requests
+portions of your testing code.")
+    (license asl2.0)))
+
+(define-public python2-requests-mock
+  (package-with-python2 python-requests-mock))
+
 (define-public python-stevedore
   (package
     (name "python-stevedore")
-- 
2.1.4

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

* Re: [PATCH 1/4] gnu: Add python-oslo.serialization.
  2015-09-14 21:23 ` [PATCH 1/4] gnu: Add python-oslo.serialization Cyril Roelandt
@ 2015-09-15 16:34   ` Mathieu Lirzin
  0 siblings, 0 replies; 16+ messages in thread
From: Mathieu Lirzin @ 2015-09-15 16:34 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

Cyril Roelandt <tipecaml@gmail.com> writes:

> * gnu/packages/openstack.scm (python-oslo.serialization,
>   python2-oslo.serialization): New variables.
> ---
>  gnu/packages/openstack.scm | 39 +++++++++++++++++++++++++++++++++++++++
>  1 file changed, 39 insertions(+)
>
> diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm
> index a9db3f2..04ddbb3 100644
> --- a/gnu/packages/openstack.scm
> +++ b/gnu/packages/openstack.scm
> @@ -290,6 +290,45 @@ in an application or library.")
>  (define-public python2-oslo.i18n
>    (package-with-python2 python-oslo.i18n))
>  
> +(define-public python-oslo.serialization
> +  (package
> +    (name "python-oslo.serialization")
> +    (version "1.9.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "https://pypi.python.org/packages/source/o/oslo.serialization/"
> +             "oslo.serialization-" version ".tar.gz"))
> +       (sha256
> +         (base32
> +          "00qaxg155s61ylh4fqc7m5fh0gijf33khhai9xvcsc9k106i3c9c"))))

   (sha256
    (base32
     "..."))

> +    (build-system python-build-system)
> +    (propagated-inputs
> +      `(("python-iso8601" ,python-iso8601)
> +        ("python-netaddr" ,python-netaddr)
> +        ("python-oslo.utils" ,python-oslo.utils)
> +        ("python-simplejson" ,python-simplejson)
> +        ("python-six" ,python-six)
> +        ("python-pytz" ,python-pytz)))
> +    (inputs
> +      `(("python-babel" ,python-babel)
> +        ("python-pbr" ,python-pbr)
> +        ("python-setuptools" ,python-setuptools)
> +        ;; Tests.
> +        ("python-mock" ,python-mock)
> +        ("python-oslo.i18n" ,python-oslo.i18n)
> +        ("python-oslotest" ,python-oslotest)))
> +    (home-page "http://launchpad.net/oslo")
> +    (synopsis "Oslo serialization library")
> +    (description
> +      "The oslo.serialization library provides support for representing objects
> +in transmittable and storable formats, such as JSON and MessagePack.")
> +    (license asl2.0)))
> +
> +(define-public python2-oslo.serialization
> +  (package-with-python2 python-oslo.serialization))
> +
>  (define-public python-oslotest
>    (package
>      (name "python-oslotest")

Otherwise, LGTM.

--
Mathieu Lirzin

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

* Re: [PATCH 2/4] gnu: Add python-oslo.context.
  2015-09-14 21:23 ` [PATCH 2/4] gnu: Add python-oslo.context Cyril Roelandt
@ 2015-09-15 16:36   ` Mathieu Lirzin
  0 siblings, 0 replies; 16+ messages in thread
From: Mathieu Lirzin @ 2015-09-15 16:36 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

Cyril Roelandt <tipecaml@gmail.com> writes:

> * gnu/packages/openstack.scm (python-oslo.context,
>   python2-oslo.context): New variables.
> ---
>  gnu/packages/openstack.scm | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm
> index 04ddbb3..1a2af79 100644
> --- a/gnu/packages/openstack.scm
> +++ b/gnu/packages/openstack.scm
> @@ -253,6 +253,37 @@ extensions.")
>  (define-public python2-oslo.config
>    (package-with-python2 python-oslo.config))
>  
> +(define-public python-oslo.context
> +  (package
> +    (name "python-oslo.context")
> +    (version "0.6.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "https://pypi.python.org/packages/source/o/oslo.context/"
> +             "oslo.context-" version ".tar.gz"))
> +       (sha256
> +         (base32
> +          "16wr9qrkc3lb94ssb14qid4liza66x316fvzjw0izg67h1a0fm86"))))

   (sha256
    (base32
     "..."))

> +    (build-system python-build-system)
> +    (inputs
> +      `(("python-babel" ,python-babel)
> +        ("python-pbr" ,python-pbr)
> +        ("python-setuptools" ,python-setuptools)
> +        ;; Tests.
> +        ("python-oslotest" ,python-oslotest)))
> +    (home-page "http://launchpad.net/oslo")
> +    (synopsis "Oslo context library")
> +    (description
> +      "The Oslo context library has helpers to maintain useful information
> +about a request context. The request context is usually populated in the WSGI
                           ^^^
2 spaces here.

> +pipeline and used by various modules such as logging.")
> +    (license asl2.0)))
> +
> +(define-public python2-oslo.context
> +  (package-with-python2 python-oslo.context))
> +
>  (define-public python-oslo.i18n
>    (package
>      (name "python-oslo.i18n")

Otherwise, LGTM.

--
Mathieu Lirzin

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

* Re: [PATCH 3/4] gnu: Add python-webob.
  2015-09-14 21:23 ` [PATCH 3/4] gnu: Add python-webob Cyril Roelandt
@ 2015-09-15 16:43   ` Mathieu Lirzin
  2015-09-15 22:58     ` Cyril Roelandt
  0 siblings, 1 reply; 16+ messages in thread
From: Mathieu Lirzin @ 2015-09-15 16:43 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

Cyril Roelandt <tipecaml@gmail.com> writes:

> * gnu/packages/python.scm (python-webob, python2-webob): New variables.
> ---
>  gnu/packages/python.scm | 27 +++++++++++++++++++++++++++
>  1 file changed, 27 insertions(+)
>
> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index c58d7f3..919573f 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -4853,3 +4853,30 @@ fractional seconds) of a clock which never goes backwards.")
>  
>  (define-public python2-monotonic
>    (package-with-python2 python-monotonic))
> +
> +(define-public python-webob
> +  (package
> +    (name "python-WebOb")
                     ^^^
Does these uppercase letters are required?

> +    (version "1.5.0b0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "https://pypi.python.org/packages/source/W/WebOb/WebOb-"
> +             version ".tar.gz"))
> +       (sha256
> +         (base32
> +          "140b3iczclk1j0405rvw5gxshqfkhcc8254fj520z3m23cwbql4a"))))

   (sha256
    (base32
     "..."))

> +    (build-system python-build-system)
> +    (inputs
> +      `(("python-nose" ,python-nose)
> +        ("python-setuptools" ,python-setuptools)))
> +    (home-page "http://webob.org/")
> +    (synopsis "WSGI request and response object")
> +    (description
> +      "WebOb provides wrappers around the WSGI request environment, and an
> +object to help create WSGI responses.")
> +    (license license:expat)))
> +
> +(define-public python2-webob
> +  (package-with-python2 python-webob))

Otherwise LGTM.

--
Mathieu Lirzin

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

* Re: [PATCH 4/4] gnu: Add python-requests-mock.
  2015-09-14 21:23 ` [PATCH 4/4] gnu: Add python-requests-mock Cyril Roelandt
@ 2015-09-15 16:54   ` Mathieu Lirzin
  2015-09-15 17:17     ` Andreas Enge
  0 siblings, 1 reply; 16+ messages in thread
From: Mathieu Lirzin @ 2015-09-15 16:54 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

Cyril Roelandt <tipecaml@gmail.com> writes:

> * gnu/packages/openstack.scm (python-requests-mock,
>   python2-requests-mock): New variables.
> ---
>  gnu/packages/openstack.scm | 31 +++++++++++++++++++++++++++++++
>  1 file changed, 31 insertions(+)
>
> diff --git a/gnu/packages/openstack.scm b/gnu/packages/openstack.scm
> index 1a2af79..2f3e883 100644
> --- a/gnu/packages/openstack.scm
> +++ b/gnu/packages/openstack.scm
> @@ -174,6 +174,37 @@ and sensible default behaviors into your setuptools run.")
>  (define-public python2-pbr
>    (package-with-python2 python-pbr))
>  
> +(define-public python-requests-mock
> +  (package
> +    (name "python-requests-mock")
> +    (version "0.6.0")
> +    (source
> +     (origin
> +       (method url-fetch)
> +       (uri (string-append
> +             "https://pypi.python.org/packages/source/r/requests-mock/"
> +             "requests-mock-" version ".tar.gz"))
> +       (sha256
> +         (base32
> +          "0gmd88c224y53b1ai8cfsrcxm9kw3gdqzysclmnaqspg7zjhxwd1"))))

   (sha256
    (base32
     "..."))

> +    (build-system python-build-system)
> +    (propagated-inputs
> +      `(("python-requests" ,python-requests)
> +        ("python-six" ,python-six)))
> +    (inputs
> +      `(("python-mock" ,python-mock)
> +        ("python-pbr" ,python-pbr)
> +        ("python-setuptools" ,python-setuptools)))
> +    (home-page "https://requests-mock.readthedocs.org/")
> +    (synopsis "Mock out responses from the requests package")
> +    (description
> +      "This module provides a building block to stub out the HTTP requests
> +portions of your testing code.")
> +    (license asl2.0)))
> +
> +(define-public python2-requests-mock
> +  (package-with-python2 python-requests-mock))
> +
>  (define-public python-stevedore
>    (package
>      (name "python-stevedore")

Otherwise LGTM.

--
Mathieu Lirzin

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

* Re: [PATCH 4/4] gnu: Add python-requests-mock.
  2015-09-15 16:54   ` Mathieu Lirzin
@ 2015-09-15 17:17     ` Andreas Enge
  2015-09-15 17:32       ` Taylan Ulrich Bayırlı/Kammer
  0 siblings, 1 reply; 16+ messages in thread
From: Andreas Enge @ 2015-09-15 17:17 UTC (permalink / raw)
  To: Mathieu Lirzin; +Cc: guix-devel

Hello,

On Tue, Sep 15, 2015 at 06:54:51PM +0200, Mathieu Lirzin wrote:
>    (sha256
>     (base32
>      "..."))

like someone else previously on this list I was also puzzled by the discussion
on formatting style (meaning, I did not understand what the indentation to
apply or to avoid is), so I wonder if there are actual rules to be formulated
and applied in case of manual formatting. My impression so far was that
indentation is usually by 2 spaces. So does this mean we now indent only by 1,
or does it depend on the context?

Andreas

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

* Re: [PATCH 4/4] gnu: Add python-requests-mock.
  2015-09-15 17:17     ` Andreas Enge
@ 2015-09-15 17:32       ` Taylan Ulrich Bayırlı/Kammer
  2015-09-15 18:36         ` Taylan Ulrich Bayırlı/Kammer
  2015-09-16 20:10         ` Andreas Enge
  0 siblings, 2 replies; 16+ messages in thread
From: Taylan Ulrich Bayırlı/Kammer @ 2015-09-15 17:32 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

Andreas Enge <andreas@enge.fr> writes:

> Hello,
>
> On Tue, Sep 15, 2015 at 06:54:51PM +0200, Mathieu Lirzin wrote:
>>    (sha256
>>     (base32
>>      "..."))
>
> like someone else previously on this list I was also puzzled by the discussion
> on formatting style (meaning, I did not understand what the indentation to
> apply or to avoid is), so I wonder if there are actual rules to be formulated
> and applied in case of manual formatting. My impression so far was that
> indentation is usually by 2 spaces. So does this mean we now indent only by 1,
> or does it depend on the context?

Emacs will use 2 spaces when the symbol (e.g. sha256 in this case) has
its 'scheme-indent-function' property set, and 1 space otherwise.  So
you get

    (let (...)
      foobar)  ;two spaces because 'let has its 'scheme-indent-function
               ;set to 1 (meaning *one* argument is "special", here the
               ;list of bindings)

and

    (display
     (string-append
      foobar
      quux))  ;one space each because they're plain functions and have
              ;no 'scheme-indent-function property set

Under Guix's sources, the .dir-locals.el file contains expressions of
the form (put 'foo 'quux value), which insert properties into symbols.
(Here the 'foo symbol gets its 'quux property set.)  Symbols appearing
there, like for instance 'with-store or 'modify-phases or 'substitute*
should probably have their arguments 2-space indented, and others
1-space, since that's what Emacs will do automatically.

By the way, shall you put a newline directly before the "special"
argument of such a form, e.g. the bindings-list of a let expression,
then you use 4 spaces for the "special" argument:

    (let
        (...)
      foobar)

    (modify-phases
        %standard-phases
      (replace 'foo bar)
      ...)

And also, as you might have noticed, it's mostly macros that get their
'scheme-indent-function set.  I don't know if this rule is enforced
under Guix.

Taylan

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

* Re: [PATCH 4/4] gnu: Add python-requests-mock.
  2015-09-15 17:32       ` Taylan Ulrich Bayırlı/Kammer
@ 2015-09-15 18:36         ` Taylan Ulrich Bayırlı/Kammer
  2015-09-16 20:10         ` Andreas Enge
  1 sibling, 0 replies; 16+ messages in thread
From: Taylan Ulrich Bayırlı/Kammer @ 2015-09-15 18:36 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

taylanbayirli@gmail.com (Taylan Ulrich "Bayırlı/Kammer") writes:

> Emacs will use 2 spaces when the symbol (e.g. sha256 in this case) has
> its 'scheme-indent-function' property set, and 1 space otherwise.  So
> you get
>
>     (let (...)
>       foobar)  ;two spaces because 'let has its 'scheme-indent-function
>                ;set to 1 (meaning *one* argument is "special", here the
>                ;list of bindings)
>
> and
>
>     (display
>      (string-append
>       foobar
>       quux))  ;one space each because they're plain functions and have
>               ;no 'scheme-indent-function property set
>
> Under Guix's sources, the .dir-locals.el file contains expressions of
> the form (put 'foo 'quux value), which insert properties into symbols.
> (Here the 'foo symbol gets its 'quux property set.)  Symbols appearing
> there, like for instance 'with-store or 'modify-phases or 'substitute*
> should probably have their arguments 2-space indented, and others
> 1-space, since that's what Emacs will do automatically.
>
> By the way, shall you put a newline directly before the "special"
> argument of such a form, e.g. the bindings-list of a let expression,
> then you use 4 spaces for the "special" argument:
>
>     (let
>         (...)
>       foobar)
>
>     (modify-phases
>         %standard-phases
>       (replace 'foo bar)
>       ...)
>
> And also, as you might have noticed, it's mostly macros that get their
> 'scheme-indent-function set.  I don't know if this rule is enforced
> under Guix.

I sent off my mail in kind of a hurry there.  I should clarify that I
was merely assuming that Emacs's default behavior would be desirable to
imitate in manual formatting, because otherwise it's pretty annoying for
Emacs users who will constantly have to be careful not to cause
whitespace changes due to Emacs's automatic behavior.  (But it's also
otherwise sensible, what Emacs does.)

The higher-ups shall say the final word. :-)

Taylan

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

* Re: [PATCH 3/4] gnu: Add python-webob.
  2015-09-15 16:43   ` Mathieu Lirzin
@ 2015-09-15 22:58     ` Cyril Roelandt
  2015-09-16  9:03       ` Andreas Enge
  0 siblings, 1 reply; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-15 22:58 UTC (permalink / raw)
  To: Mathieu Lirzin; +Cc: guix-devel

On 09/15/2015 06:43 PM, Mathieu Lirzin wrote:
> Cyril Roelandt <tipecaml@gmail.com> writes:
>> > +
>> > +(define-public python-webob
>> > +  (package
>> > +    (name "python-WebOb")
>                      ^^^
> Does these uppercase letters are required?
> 

Well, that is a good question. I think we currently use lowercase
letters for the whole name, even though upstream does things differently
(see python-babel, for instance, which is spelled "Babel" upstream).
Maybe the wise thing to do is to keep doing this, so I'll use
"python-webob".

Cyril.

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

* Re: [PATCH 3/4] gnu: Add python-webob.
  2015-09-15 22:58     ` Cyril Roelandt
@ 2015-09-16  9:03       ` Andreas Enge
  0 siblings, 0 replies; 16+ messages in thread
From: Andreas Enge @ 2015-09-16  9:03 UTC (permalink / raw)
  To: Cyril Roelandt; +Cc: guix-devel

On Wed, Sep 16, 2015 at 12:58:12AM +0200, Cyril Roelandt wrote:
> >> > +    (name "python-WebOb")
> Maybe the wise thing to do is to keep doing this, so I'll use
> "python-webob".

See the section "package naming" in the manual:
   https://www.gnu.org/software/guix/manual/guix.html#Package-Naming
"Both (...) correspond to the lowercase conversion of the project name chosen
upstream".

Andreas

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

* Re: [PATCH 4/4] gnu: Add python-requests-mock.
  2015-09-15 17:32       ` Taylan Ulrich Bayırlı/Kammer
  2015-09-15 18:36         ` Taylan Ulrich Bayırlı/Kammer
@ 2015-09-16 20:10         ` Andreas Enge
  1 sibling, 0 replies; 16+ messages in thread
From: Andreas Enge @ 2015-09-16 20:10 UTC (permalink / raw)
  To: Taylan Ulrich Bayırlı/Kammer; +Cc: guix-devel

Thanks, Taylan, for your explanations! Needing to consult a list of
functions in a file to get the rules is not as conceptual as I would have
hoped for; but as far as I can see, it means that most of the time indenting
by 1 is the desired thing. I will take this into account for my next packages.

Andreas

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

* Re: [PATCH 0/4] Add more dependencies of OpenStack.
  2015-09-14 21:23 [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
                   ` (3 preceding siblings ...)
  2015-09-14 21:23 ` [PATCH 4/4] gnu: Add python-requests-mock Cyril Roelandt
@ 2015-09-16 20:54 ` Cyril Roelandt
  4 siblings, 0 replies; 16+ messages in thread
From: Cyril Roelandt @ 2015-09-16 20:54 UTC (permalink / raw)
  To: guix-devel

On 09/14/2015 11:23 PM, Cyril Roelandt wrote:
> These packages would be really useful to have for a demo of guix-tox :)
> 

Pushed, thanks for the reviews!

Cyril.

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

end of thread, other threads:[~2015-09-16 20:54 UTC | newest]

Thread overview: 16+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2015-09-14 21:23 [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt
2015-09-14 21:23 ` [PATCH 1/4] gnu: Add python-oslo.serialization Cyril Roelandt
2015-09-15 16:34   ` Mathieu Lirzin
2015-09-14 21:23 ` [PATCH 2/4] gnu: Add python-oslo.context Cyril Roelandt
2015-09-15 16:36   ` Mathieu Lirzin
2015-09-14 21:23 ` [PATCH 3/4] gnu: Add python-webob Cyril Roelandt
2015-09-15 16:43   ` Mathieu Lirzin
2015-09-15 22:58     ` Cyril Roelandt
2015-09-16  9:03       ` Andreas Enge
2015-09-14 21:23 ` [PATCH 4/4] gnu: Add python-requests-mock Cyril Roelandt
2015-09-15 16:54   ` Mathieu Lirzin
2015-09-15 17:17     ` Andreas Enge
2015-09-15 17:32       ` Taylan Ulrich Bayırlı/Kammer
2015-09-15 18:36         ` Taylan Ulrich Bayırlı/Kammer
2015-09-16 20:10         ` Andreas Enge
2015-09-16 20:54 ` [PATCH 0/4] Add more dependencies of OpenStack Cyril Roelandt

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