unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
* [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
@ 2016-02-08 16:55 Pjotr Prins
  2016-02-08 17:26 ` Christopher Allan Webber
  2016-02-08 20:06 ` Andreas Enge
  0 siblings, 2 replies; 10+ messages in thread
From: Pjotr Prins @ 2016-02-08 16:55 UTC (permalink / raw)
  To: guix-devel

* gnu/packages/python.scm (python-sqlalchemy):
Update to 1.0.11
---
 gnu/packages/python.scm | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
index 5b988c1..47c7975 100644
--- a/gnu/packages/python.scm
+++ b/gnu/packages/python.scm
@@ -3549,7 +3549,7 @@ routines such as routines for numerical integration and optimization.")
 (define-public python-sqlalchemy
   (package
     (name "python-sqlalchemy")
-    (version "0.9.7")
+    (version "1.0.11")
     (source
      (origin
       (method url-fetch)
@@ -3557,7 +3557,7 @@ routines such as routines for numerical integration and optimization.")
                           "SQLAlchemy/SQLAlchemy-" version ".tar.gz"))
       (sha256
        (base32
-        "059ayifj5l08v6vv56anhyibyllscn10dlzr2fcw68gz1hfjdzsz"))))
+        "18z81m3ypcgqmsqn61qpfm75b98z5fihx23hjw05a57shybp490b"))))
     (build-system python-build-system)
     (native-inputs
      `(("python-cython" ,python-cython) ;for c extensions
-- 
2.1.4

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 16:55 [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name) Pjotr Prins
@ 2016-02-08 17:26 ` Christopher Allan Webber
  2016-02-08 19:46   ` Leo Famulari
  2016-02-08 20:06 ` Andreas Enge
  1 sibling, 1 reply; 10+ messages in thread
From: Christopher Allan Webber @ 2016-02-08 17:26 UTC (permalink / raw)
  To: Pjotr Prins; +Cc: guix-devel

Pjotr Prins writes:

> * gnu/packages/python.scm (python-sqlalchemy):
> Update to 1.0.11
> ---
>  gnu/packages/python.scm | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/gnu/packages/python.scm b/gnu/packages/python.scm
> index 5b988c1..47c7975 100644
> --- a/gnu/packages/python.scm
> +++ b/gnu/packages/python.scm
> @@ -3549,7 +3549,7 @@ routines such as routines for numerical integration and optimization.")
>  (define-public python-sqlalchemy
>    (package
>      (name "python-sqlalchemy")
> -    (version "0.9.7")
> +    (version "1.0.11")
>      (source
>       (origin
>        (method url-fetch)
> @@ -3557,7 +3557,7 @@ routines such as routines for numerical integration and optimization.")
>                            "SQLAlchemy/SQLAlchemy-" version ".tar.gz"))
>        (sha256
>         (base32
> -        "059ayifj5l08v6vv56anhyibyllscn10dlzr2fcw68gz1hfjdzsz"))))
> +        "18z81m3ypcgqmsqn61qpfm75b98z5fihx23hjw05a57shybp490b"))))
>      (build-system python-build-system)
>      (native-inputs
>       `(("python-cython" ,python-cython) ;for c extensions

LGTM, push it!

. o O (iirc, version bump pushes are often ok to just push to guix)
      (master if you have commit access without review, right?)

 - Chris

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 17:26 ` Christopher Allan Webber
@ 2016-02-08 19:46   ` Leo Famulari
  2016-02-08 23:03     ` Christopher Allan Webber
  0 siblings, 1 reply; 10+ messages in thread
From: Leo Famulari @ 2016-02-08 19:46 UTC (permalink / raw)
  To: Christopher Allan Webber; +Cc: guix-devel

On Mon, Feb 08, 2016 at 09:26:42AM -0800, Christopher Allan Webber wrote:
> Pjotr Prins writes:
> 
> > * gnu/packages/python.scm (python-sqlalchemy):
> > Update to 1.0.11

[...]

> LGTM, push it!
> 
> . o O (iirc, version bump pushes are often ok to just push to guix)
>       (master if you have commit access without review, right?)

I'd say they're fine to push if they are a leaf package or a seldom used
library. On the other hand, if many things depend on them, it's best to
ask, or push to a staging branch if you already know that's the answer.

> 
>  - Chris
> 

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 16:55 [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name) Pjotr Prins
  2016-02-08 17:26 ` Christopher Allan Webber
@ 2016-02-08 20:06 ` Andreas Enge
  2016-02-08 20:07   ` Andreas Enge
  1 sibling, 1 reply; 10+ messages in thread
From: Andreas Enge @ 2016-02-08 20:06 UTC (permalink / raw)
  To: Pjotr Prins; +Cc: guix-devel

Hello,

I applied the patch and got a test failure:

sqlalchemy.exc.NoSuchColumnError: "Could not locate column in row for column 'users.user_id'"
=========================== short test summary info ============================
FAIL test/sql/test_resultset.py::ResultProxyTest_sqlite_pysqlite::()::test_column_accessor_sqlite_raw
FAIL test/sql/test_resultset.py::ResultProxyTest_sqlite_pysqlite::()::test_column_accessor_sqlite_translated
============= 2 failed, 6531 passed, 785 skipped in 194.43 seconds =============
phase `check' failed after 200.1 seconds

The same for the Python 2 version.

Andreas

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 20:06 ` Andreas Enge
@ 2016-02-08 20:07   ` Andreas Enge
  2016-02-08 20:49     ` Pjotr Prins
  0 siblings, 1 reply; 10+ messages in thread
From: Andreas Enge @ 2016-02-08 20:07 UTC (permalink / raw)
  To: Pjotr Prins; +Cc: guix-devel

Sorry, I did not realise that this is related to the sqlite thing.
Maybe file a bug report upstream?

Andreas

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 20:07   ` Andreas Enge
@ 2016-02-08 20:49     ` Pjotr Prins
  2016-02-08 20:57       ` Andreas Enge
  0 siblings, 1 reply; 10+ messages in thread
From: Pjotr Prins @ 2016-02-08 20:49 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

On Mon, Feb 08, 2016 at 09:07:46PM +0100, Andreas Enge wrote:
> Sorry, I did not realise that this is related to the sqlite thing.

Yes, it is not about this version of python-sqlalchemy.

> Maybe file a bug report upstream?

They will fix it when they upgrade sqlite. python-sqlalchemy is under
active development. Question is what we do for the time being with
sqlite. 

Pj.

-- 

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 20:49     ` Pjotr Prins
@ 2016-02-08 20:57       ` Andreas Enge
  2016-02-08 21:09         ` Pjotr Prins
  0 siblings, 1 reply; 10+ messages in thread
From: Andreas Enge @ 2016-02-08 20:57 UTC (permalink / raw)
  To: Pjotr Prins; +Cc: guix-devel

On Mon, Feb 08, 2016 at 09:49:51PM +0100, Pjotr Prins wrote:
> They will fix it when they upgrade sqlite. python-sqlalchemy is under
> active development. Question is what we do for the time being with
> sqlite. 

I would say the culprit is python-sqlalchemy and not sqlite. So if you
cannot wait for upstream to fix their package, one possibility would be
to add a package for on older version of sqlite (assuming it does not
present security problems) and to build python-sqlalchemy with it.
It could even be a private variable only used in this input instead of
a user visible variant.

Andreas

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 20:57       ` Andreas Enge
@ 2016-02-08 21:09         ` Pjotr Prins
  2016-02-10 16:51           ` Efraim Flashner
  0 siblings, 1 reply; 10+ messages in thread
From: Pjotr Prins @ 2016-02-08 21:09 UTC (permalink / raw)
  To: Andreas Enge; +Cc: guix-devel

Yes, that is what I have already done. I can't afford to wait ;)

I am just wondering how many packages are broken (visibly or not).

Sqlite changed something fundamental.

Pj.

On Mon, Feb 08, 2016 at 09:57:10PM +0100, Andreas Enge wrote:
> On Mon, Feb 08, 2016 at 09:49:51PM +0100, Pjotr Prins wrote:
> > They will fix it when they upgrade sqlite. python-sqlalchemy is under
> > active development. Question is what we do for the time being with
> > sqlite. 
> 
> I would say the culprit is python-sqlalchemy and not sqlite. So if you
> cannot wait for upstream to fix their package, one possibility would be
> to add a package for on older version of sqlite (assuming it does not
> present security problems) and to build python-sqlalchemy with it.
> It could even be a private variable only used in this input instead of
> a user visible variant.
> 
> Andreas
> 

-- 

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 19:46   ` Leo Famulari
@ 2016-02-08 23:03     ` Christopher Allan Webber
  0 siblings, 0 replies; 10+ messages in thread
From: Christopher Allan Webber @ 2016-02-08 23:03 UTC (permalink / raw)
  To: Leo Famulari; +Cc: guix-devel

Leo Famulari writes:

> On Mon, Feb 08, 2016 at 09:26:42AM -0800, Christopher Allan Webber wrote:
>> Pjotr Prins writes:
>> 
>> > * gnu/packages/python.scm (python-sqlalchemy):
>> > Update to 1.0.11
>
> [...]
>
>> LGTM, push it!
>> 
>> . o O (iirc, version bump pushes are often ok to just push to guix)
>>       (master if you have commit access without review, right?)
>
> I'd say they're fine to push if they are a leaf package or a seldom used
> library. On the other hand, if many things depend on them, it's best to
> ask, or push to a staging branch if you already know that's the answer.

Thanks for the clarity!

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

* Re: [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name).
  2016-02-08 21:09         ` Pjotr Prins
@ 2016-02-10 16:51           ` Efraim Flashner
  0 siblings, 0 replies; 10+ messages in thread
From: Efraim Flashner @ 2016-02-10 16:51 UTC (permalink / raw)
  To: Pjotr Prins; +Cc: guix-devel

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

On Mon, 8 Feb 2016 22:09:25 +0100
Pjotr Prins <pjotr.public12@thebird.nl> wrote:

> Yes, that is what I have already done. I can't afford to wait ;)
> 
> I am just wondering how many packages are broken (visibly or not).
> 
> Sqlite changed something fundamental.
> 
> Pj.
> 

I locally updated sqlite to 3.10.2 and that didn't help with
python-sqlalchemy.

-- 
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: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 819 bytes --]

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

end of thread, other threads:[~2016-02-10 16:51 UTC | newest]

Thread overview: 10+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2016-02-08 16:55 [PATCH] gnu: python-sqlalchemy: Update to 1.0.11 (oops: fixed name) Pjotr Prins
2016-02-08 17:26 ` Christopher Allan Webber
2016-02-08 19:46   ` Leo Famulari
2016-02-08 23:03     ` Christopher Allan Webber
2016-02-08 20:06 ` Andreas Enge
2016-02-08 20:07   ` Andreas Enge
2016-02-08 20:49     ` Pjotr Prins
2016-02-08 20:57       ` Andreas Enge
2016-02-08 21:09         ` Pjotr Prins
2016-02-10 16:51           ` Efraim Flashner

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