[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
[bug#33099] [PATCH] I add r-aspi package appended to cran.scm file
From: |
Laura Lazzati |
Subject: |
[bug#33099] [PATCH] I add r-aspi package appended to cran.scm file |
Date: |
Mon, 22 Oct 2018 19:29:41 -0300 |
On Mon, Oct 22, 2018 at 10:39 AM Gábor Boskovits <address@hidden> wrote:
>
> Hi Laura,
>
> Laura Lazzati <address@hidden> ezt írta (időpont: 2018.
> okt. 22., H, 14:59):
> >
> > On Mon, 22 Oct 2018, 05:47 Gábor Boskovits, <address@hidden> wrote:
> > >
> > > Hello Laura,
> > >
> > > Almost there!
> >
> > Yes, after trying lots of things I figured out why it took me so long
> > to package it. The short story is that I did not understand that guix
> > import cran -r aspi was creating the template for me, I thought it
> > worked only for packages that were already packeged.I applied it to
> > another r package already available in guix, and modified it with
> > information about aspi from its website. The good side is that I
> > learned tons of guix commands.
> > >
> > >
> > > Laura Lazzati <address@hidden> ezt írta (időpont: 2018.
> > > okt. 21., V, 23:54):
> > > >
> > > > From 3628e4d7d4814777bb748c3f0ebba133814b2b94 Mon Sep 17 00:00:00 2001
> > > > From: laura <address@hidden>
> > > > Date: Sun, 21 Oct 2018 18:38:58 -0300
> > > > Subject: [PATCH] gnu: Add r-aspi
> > > >
> > > > * gnu/packages/cran.scm (r-aspi): New variable
> > > > ---
> > > > gnu/packages/cran.scm | 21 +++++++++++++++++++++
> > > > 1 file changed, 21 insertions(+)
> > > >
> > >
> > > Just a bit knitpicking, could you add the periods to the commit message,
> > > like:
> > >
> > > gnu: Add r-aspi.
> > >
> > > * gnu/packages/patches/cran.scm (r-aspi): New variable.
> > >
> > >
> > > > diff --git a/gnu/packages/cran.scm b/gnu/packages/cran.scm
> > > > index efcc2dccb..c219065e8 100644
> > > > --- a/gnu/packages/cran.scm
> > > > +++ b/gnu/packages/cran.scm
> > > > @@ -6256,3 +6256,24 @@ a header row or not. The dataset should
> > > > contain only one independent variable
> > > > (x) and one dependent variable (y) and can contain a weight for each
> > > > observation.")
> > > > (license license:gpl2)))
> > > > +
> > > > +(define-public r-aspi
> > > > + (package
> > > > + (name "r-aspi")
> > > > + (version "0.2.0")
> > > > + (source
> > > > + (origin
> > > > + (method url-fetch)
> > > > + (uri (cran-uri "aspi" version))
> > > > + (sha256
> > > > + (base32
> > > > + "0rhvxw243vvdv3hxa6pi343gcjc2cbxq1jzqirl9k1l4i3897l87"))))
> > > > + (build-system r-build-system)
> > > > + (home-page
> > > > + "http://cran.r-project.org/web/packages/aspi")
> > > > + (synopsis
> > > > + "Analysis of symmetry of parasitic infections")
> > > > + (description
> > > > + "This package provides tools for the analysis and visualization
> > > > of bilateral
> > > > + asymmetry in parasitic infections.")
> > > > + (license license:gpl3+)))
> > > > --
> > > > 2.17.1
> > > >
> > >
> > > This is fine now.
> > >
> > > I still see these strange line breaks in the patch, how does that look on
> > > you side?
> >
> > I guess the problem was that I was adding a line break because in the
> > previous patch when building I had a complain that that line was too
> > long. I have just told emacs to truncate long lines and buld the
> > package without problems. My fault. It was one of the two changes that
> > I made from the original definiton. Adding that break line and adding
> > licence: before gpl3+ because otherwhise I got the error: Unbound
> > variable: gpl3+.
> > >
> > >
> > > Can you apply you own patches using git am?
> > In my local repo?
> > >
> > > You can send an updated patch with the corrected commit message,
> > > or I can apply these minor changes and push it.
> > > WDYT?
I have already changed the commit messages.
I have generated several times the new patch, went back and so forth.
I can't figure out what is wrong.
I tried applying the different patches I went on generating with git
am <anyaspipatch>.patch
I tend to get this error: error: patch failed: gnu/packages/cran.scm:6256
And when I take a look at cran.scm, that line belongs to the previous
package (the one before appending mines)
I read in the coding style that lines must not exceed 80 columns, but
when I open cran.scm there are several packages that do so.
And if I don't add an eol in the description (is it there where you
see the line break?), after running lint command I get:
/home/laura/guix/gnu/packages/cran.scm:6261:2: address@hidden: line 6276
is way too long (119 characters)
Any ideas of how to help me fix this?
> > If it is not a problem for you, let me try :)
> > Do you mean by changing the subject of this chain of messages?
>
> Yes, that would do half the job.
>
> There is also a missing period in this line:
> * gnu/packages/cran.scm (r-aspi): New variable
>
> You can either fix these up manually, or using
> git commit --amend, and fix these in the commit message.
> That way git format-patch will fix these up.
>
> > Is that
> > OK for sending the patch with the corrections, but keeping the
> > debugs.gnu.org mail with the number as it is in the To field?
>
> Yes that is fine.
>
> > >
> > > Best regards,
> > > g_bor
> >
> > Regards :)
> > Laura
>
> Best regards,
> g_bor
Regards!
Laura
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, (continued)
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Gábor Boskovits, 2018/10/20
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Gábor Boskovits, 2018/10/20
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Laura Lazzati, 2018/10/20
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Ricardo Wurmus, 2018/10/20
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Laura Lazzati, 2018/10/20
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Laura Lazzati, 2018/10/21
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Laura Lazzati, 2018/10/21
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Gábor Boskovits, 2018/10/22
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Laura Lazzati, 2018/10/22
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Gábor Boskovits, 2018/10/22
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file,
Laura Lazzati <=
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Ricardo Wurmus, 2018/10/23
- [bug#33099] [PATCH] I add r-aspi package appended to cran.scm file, Gábor Boskovits, 2018/10/23
[bug#33099] gnu: Add r-aspi., Laura Lazzati, 2018/10/23
- [bug#33099] gnu: Add r-aspi., Björn Höfling, 2018/10/23
- [bug#33099] gnu: Add r-aspi., Laura Lazzati, 2018/10/23
- [bug#33099] gnu: Add r-aspi., Gábor Boskovits, 2018/10/23
- [bug#33099] gnu: Add r-aspi., Björn Höfling, 2018/10/23
- [bug#33099] gnu: Add r-aspi., Laura Lazzati, 2018/10/23
- [bug#33099] gnu: Add r-aspi., Björn Höfling, 2018/10/23
- bug#33099: gnu: Add r-aspi., Gábor Boskovits, 2018/10/23