bug#47972: closed (Add irods.)

[GNU bug Tracking System]

Your message dated Mon, 14 Jun 2021 16:48:29 +0200
with message-id <87pmwoqz7m.fsf@elephly.net>
and subject line Re: bug#47972: Add irods.
has caused the debbugs.gnu.org bug report #47972,
regarding Add irods.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
47972: http://debbugs.gnu.org/cgi/bugreport.cgi?bug=47972
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
> --- Begin Message ---
>
>
>
> Subject: 
>
> Add irods.
>
>
>
>
> Date: 
>
> Fri, 23 Apr 2021 16:20:04 +0200
>
>
>
>
> User-agent: 
>
> mu4e 1.4.15; emacs 27.2
>
>
>
>
> Hi Guix,
>
> this set of patches adds irods and irods-client-icommands.  What 
> makes this patch set a little annoying is the fact that irods 
> insists on being built with clang 6; this means that we need 
> variants of existing GCC-built packages just so that they can be 
> linked with the irods libraries.
>
> For the most part this is not too bad because we need specific 
> versions of certain libraries anyway.  But for snappy-with-clang6 
> and avro-cpp it’s pretty bad.  You’ll see that I’m using an inline 
> package definition for a variant of avro-cpp that is built with 
> clang; that package variant needs snappy, so I had to introduce 
> snappy-with-clang6.
>
> Before we can actually merge this we would have to move this 
> avro-cpp variant to its own top-level package definition, because 
> it turns out that irods-client-icommands also needs it.
>
> If you have an idea how to build irods with GCC, I’d be very 
> happy.  (It takes quite some time to build irods, so be prepared 
> for long iteration cycles…)
>
> --
> Ricardo
>
>
>
> --- End Message ---
> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#47972: Add irods.
>
>
>
>
> Date: 
>
> Mon, 14 Jun 2021 16:48:29 +0200
>
>
>
>
> User-agent: 
>
> mu4e 1.4.15; emacs 27.2
>
>
>
>
>
> Ludovic Courtès <ludovic.courtes@inria.fr> writes:
>
> > Hi Ricardo,
> >
> > Ricardo Wurmus <rekado@elephly.net> skribis:
> >
> > > * gnu/packages/boost.scm (boost-for-irods): New variable.
> >
> > [...]
> >
> > > +;; Sadly, this is needed for irods.  It won't link with 1.69 
> > > or later.
> > > +(define-public boost-for-irods
> > > +  (package
> > > +    (name "boost-for-irods")
> > > +    (version "1.68.0")
> >
> > For this and the other “for-irods” packages, how about:
> >
> >   1. Inheriting from the original package (‘boost’ in this 
> >   case), to at
> >      least avoid duplicating the synopsis, description, and 
> >      license?
> >
> >   2. Adding the ‘hidden?’ property, because they are not meant 
> >   to be
> >      installed on their own?
>
> Done.
>
> I also addressed what I could of Maxime’s comments.  Thank you.
>
> I split out the package variant of avro-cpp that is used by both 
> irods packages and pushed the patch series to the “master” branch.
>
>
> --
> Ricardo
>
>
> --- End Message ---