Possible circular dependecy about sparqlwrapper HOT 10 CLOSED

they depend on each other as some of their features require the other lib...

pip will resolve this dependency when installing, resulting in both packages to be installed no matter which one you want.

not sure if this makes sense, but we're always open to suggestions ;)

from sparqlwrapper.

That is OK for local installation using pip. But conda (and other packaging systems) do not allow downloads during build time. I am trying to get in touch with linux packagers to see how they work around that.

I am not sure what is the best way to go, never saw a circular (hard) dependency before. As for soft dependencies, a simple patch removing it would be sufficient.

from sparqlwrapper.

My 2 cents: Stop (falsely) treating them as separate packages. Make a clean break and combine them into a single source tree. Alternative: when I've had this issue in Java, I would abstract out interfaces to depend on, then use dependency injection in initialization.

from sparqlwrapper.

This is related to RDFLib/rdflib#391.

from sparqlwrapper.

yupp... do we want to keep the discussion there?

Thing is that until now no one stepped ahead pulling out functionality into something like rdflib-core and it's probably not that small a change at all :-/

from sparqlwrapper.

@joernhees up to you. Feel free to close this issue. I am following RDFLib/rdflib#391 now.

As a packager I believe that the current approach is broken. No packaging system allows "free will" for pip to download while creating the binaries. I investigated how some Linux distros did it and I found out that, earlier versions of rdflib and sparqlwrapper did not have the circular dependency issue, so they had a version of each other to download when the upgrade happened.

from sparqlwrapper.

ok, closing this in favor of RDFLib/rdflib#391

from sparqlwrapper.

Do you mean RDFLib/rdflb#391?

-- Sent from my phone. Please forgive brevity.

From: Jörn Heesmailto:[email protected]
Sent: ‎10/‎27/‎2014 10:58 AM
To: RDFLib/sparqlwrappermailto:[email protected]
Cc: Dale Vissermailto:[email protected]
Subject: Re: [sparqlwrapper] Possible circular dependecy (#46)

ok, closing this in favor of RDFLib/rdflib#39

Reply to this email directly or view it on GitHub:
#46 (comment)

from sparqlwrapper.

;) thx, changed

from sparqlwrapper.

I guess I arrived too late to this thread(s)...

Would help to move RDFLib to extras_require also here?

from sparqlwrapper.