GNU bug report logs - #30894
Submission of patch for Add r-abc

Previous Next

Package: guix-patches;

Reported by: Sahithi Yarlagadda <sahi <at> swecha.net>

Date: Wed, 21 Mar 2018 11:17:02 UTC

Severity: normal

Tags: moreinfo

Done: Ricardo Wurmus <rekado <at> elephly.net>

Bug is archived. No further changes may be made.

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 30894 in the body.
You can then email your comments to 30894 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Wed, 21 Mar 2018 11:17:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Sahithi Yarlagadda <sahi <at> swecha.net>:
New bug report received and forwarded. Copy sent to guix-patches <at> gnu.org. (Wed, 21 Mar 2018 11:17:02 GMT) Full text and rfc822 format available.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Sahithi Yarlagadda <sahi <at> swecha.net>
To: guix-patches <at> gnu.org
Cc: Ricardo Wurmus <rekado <at> elephly.net>
Subject: Submission of patch for Add r-abc
Date: Wed, 21 Mar 2018 16:44:59 +0530
[Message part 1 (text/plain, inline)]
Hi,

I am submitting a patch for r-abc. Please review.

-- 
Regards
Sahithi

[0001-gnu-Add-r-abc.patch (text/x-patch, attachment)]

Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Wed, 21 Mar 2018 21:29:02 GMT) Full text and rfc822 format available.

Message #8 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Sahithi Yarlagadda <sahi <at> swecha.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: Submission of patch for Add r-abc
Date: Wed, 21 Mar 2018 22:27:49 +0100
Hi Sahithi,

> I am submitting a patch for r-abc. Please review.

Thank you for the patch!

What follows is a short review that includes things I could change
myself before applying the patch, but for the sake of learning I
explicitly point out in this message.  I hope you don’t mind that I’m
focusing on what may seem like minor details.

> From 4b3b3eee4995616059d8a27489affc43cee26ea4 Mon Sep 17 00:00:00 2001
> From: Sahithi Yarlagadda <sahi <at> swecha.net>
> Date: Wed, 21 Mar 2018 16:30:18 +0530
> Subject: [PATCH] gnu: Add r-abc.
>
>     * gnu/packages/cran.scm (r-abc): New variable.

Nitpick: please don’t indent this line.  The asterisk should be the
first character in this line.

> +
> +(define-public r-abc
> +  (package
> +    (name "r-abc")
> +    (version "2.1")
> +    (source
> +      (origin
> +        (method url-fetch)
> +        (uri (cran-uri "abc" version))
> +        (sha256
> +          (base32
> +            "0ngzaaz2y2s03fhngvwipmy4kq38xrmyddaz6a6l858rxvadrlhb"))))
> +    (build-system r-build-system)
> +    (propagated-inputs
> +      `(("r-abc-data" ,r-abc-data)
> +        ("r-locfit" ,r-locfit)
> +        ("r-mass" ,r-mass)
> +        ("r-nnet" ,r-nnet)
> +        ("r-quantreg" ,r-quantreg)))
> +    (home-page
> +      "http://cran.r-project.org/web/packages/abc")

Please pull this onto the previous line and use “https”.

> +    (synopsis
> +      "Tools for approximate bayesian computation (ABC)")

Please combine this line with the previous line.

> +    (description
> +      "Implements several ABC algorithms for performing parameter estimation,
> +model selection, and goodness-of-fit.  Cross-validation tools are also
> +available for measuring the accuracy of ABC estimates, and to calculate the
> +misclassification probabilities of different models.")

Please use full sentences.  For CRAN packages the official description
usually provides only a sentence fragment in place of the first
sentence.  Please also use “@dfn{Approximate Bayesian Computation}
(ABC)” the first time “ABC” is mentioned.

All thoughout the indentation is a bit off (that’s expected as the
importer is very simple and doesn’t know about the expected
indentation).  If you’re using Emacs you can easily adjust it with M-C-q
(also known as “indent-sexp”); otherwise please use
“etc/indent-code.el”.

Have you built this package with Guix?

Finally, please rebase this patch so that it can be applied
independently from the other patch you’ve sent, which cannot currently
be applied due to problems in building the defined package.  Rebasing
this patch ensures that it can be merged even if the other patch is
stuck.

Thanks!

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net






Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Wed, 28 Mar 2018 19:46:01 GMT) Full text and rfc822 format available.

Message #11 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Sahithi Yarlagadda <sahi <at> swecha.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Wed, 28 Mar 2018 21:44:26 +0200
Hi Sahiti,

have you been able to look at my review below?
Please let me know if anything is unclear.

~~ Ricardo

Ricardo Wurmus <rekado <at> elephly.net> writes:

> Hi Sahithi,
>
>> I am submitting a patch for r-abc. Please review.
>
> Thank you for the patch!
>
> What follows is a short review that includes things I could change
> myself before applying the patch, but for the sake of learning I
> explicitly point out in this message.  I hope you don’t mind that I’m
> focusing on what may seem like minor details.
>
>> From 4b3b3eee4995616059d8a27489affc43cee26ea4 Mon Sep 17 00:00:00 2001
>> From: Sahithi Yarlagadda <sahi <at> swecha.net>
>> Date: Wed, 21 Mar 2018 16:30:18 +0530
>> Subject: [PATCH] gnu: Add r-abc.
>>
>>     * gnu/packages/cran.scm (r-abc): New variable.
>
> Nitpick: please don’t indent this line.  The asterisk should be the
> first character in this line.
>
>> +
>> +(define-public r-abc
>> +  (package
>> +    (name "r-abc")
>> +    (version "2.1")
>> +    (source
>> +      (origin
>> +        (method url-fetch)
>> +        (uri (cran-uri "abc" version))
>> +        (sha256
>> +          (base32
>> +            "0ngzaaz2y2s03fhngvwipmy4kq38xrmyddaz6a6l858rxvadrlhb"))))
>> +    (build-system r-build-system)
>> +    (propagated-inputs
>> +      `(("r-abc-data" ,r-abc-data)
>> +        ("r-locfit" ,r-locfit)
>> +        ("r-mass" ,r-mass)
>> +        ("r-nnet" ,r-nnet)
>> +        ("r-quantreg" ,r-quantreg)))
>> +    (home-page
>> +      "http://cran.r-project.org/web/packages/abc")
>
> Please pull this onto the previous line and use “https”.
>
>> +    (synopsis
>> +      "Tools for approximate bayesian computation (ABC)")
>
> Please combine this line with the previous line.
>
>> +    (description
>> +      "Implements several ABC algorithms for performing parameter estimation,
>> +model selection, and goodness-of-fit.  Cross-validation tools are also
>> +available for measuring the accuracy of ABC estimates, and to calculate the
>> +misclassification probabilities of different models.")
>
> Please use full sentences.  For CRAN packages the official description
> usually provides only a sentence fragment in place of the first
> sentence.  Please also use “@dfn{Approximate Bayesian Computation}
> (ABC)” the first time “ABC” is mentioned.
>
> All thoughout the indentation is a bit off (that’s expected as the
> importer is very simple and doesn’t know about the expected
> indentation).  If you’re using Emacs you can easily adjust it with M-C-q
> (also known as “indent-sexp”); otherwise please use
> “etc/indent-code.el”.
>
> Have you built this package with Guix?
>
> Finally, please rebase this patch so that it can be applied
> independently from the other patch you’ve sent, which cannot currently
> be applied due to problems in building the defined package.  Rebasing
> this patch ensures that it can be merged even if the other patch is
> stuck.
>
> Thanks!






Added tag(s) moreinfo. Request was from Ricardo Wurmus <rekado <at> elephly.net> to control <at> debbugs.gnu.org. (Wed, 28 Mar 2018 19:52:02 GMT) Full text and rfc822 format available.

Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Thu, 29 Mar 2018 10:16:02 GMT) Full text and rfc822 format available.

Message #16 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Sahithi Yarlagadda <sahi <at> swecha.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Thu, 29 Mar 2018 15:45:19 +0530
[Message part 1 (text/plain, inline)]
Hi Ricardo


I have a small doubt regarding the build. Let me explain the process i
am doing the build.

1) Cloning the Source

2) Running guix environment guix

3) ./bootstrap

4) ./configure --localstatedir=/var

5) make

6) Make Changes to the cran.scm file

7) Build using  ./pre-inst-env guix build <package>

8) If build is success, format the patch and submit the patch

*When i am planning to write a different package description, i am
following this*

9) rebasing the patch

10) Pull for any changes

11) running the process from (2)


Am i doing the right way, I hope i have to compile everytime and then
make changes each Package Description. Because when i tired to just make
changes and build on old one, i got many errors and warning saying the
.go file is older than the .scm.


Please suggest me.


On Thursday 29 March 2018 01:14 AM, Ricardo Wurmus wrote:
> Hi Sahiti,
>
> have you been able to look at my review below?

> Please let me know if anything is unclear.
>
> ~~ Ricardo
>
> Ricardo Wurmus <rekado <at> elephly.net> writes:
>
>> Hi Sahithi,
>>
>>> I am submitting a patch for r-abc. Please review.
>> Thank you for the patch!
>>
>> What follows is a short review that includes things I could change
>> myself before applying the patch, but for the sake of learning I
>> explicitly point out in this message.  I hope you don’t mind that I’m
>> focusing on what may seem like minor details.
>>
>>> From 4b3b3eee4995616059d8a27489affc43cee26ea4 Mon Sep 17 00:00:00 2001
>>> From: Sahithi Yarlagadda <sahi <at> swecha.net>
>>> Date: Wed, 21 Mar 2018 16:30:18 +0530
>>> Subject: [PATCH] gnu: Add r-abc.
>>>
>>>     * gnu/packages/cran.scm (r-abc): New variable.
>> Nitpick: please don’t indent this line.  The asterisk should be the
>> first character in this line.
>>
>>> +
>>> +(define-public r-abc
>>> +  (package
>>> +    (name "r-abc")
>>> +    (version "2.1")
>>> +    (source
>>> +      (origin
>>> +        (method url-fetch)
>>> +        (uri (cran-uri "abc" version))
>>> +        (sha256
>>> +          (base32
>>> +            "0ngzaaz2y2s03fhngvwipmy4kq38xrmyddaz6a6l858rxvadrlhb"))))
>>> +    (build-system r-build-system)
>>> +    (propagated-inputs
>>> +      `(("r-abc-data" ,r-abc-data)
>>> +        ("r-locfit" ,r-locfit)
>>> +        ("r-mass" ,r-mass)
>>> +        ("r-nnet" ,r-nnet)
>>> +        ("r-quantreg" ,r-quantreg)))
>>> +    (home-page
>>> +      "http://cran.r-project.org/web/packages/abc")
>> Please pull this onto the previous line and use “https”.
>>
>>> +    (synopsis
>>> +      "Tools for approximate bayesian computation (ABC)")
>> Please combine this line with the previous line.
>>
>>> +    (description
>>> +      "Implements several ABC algorithms for performing parameter estimation,
>>> +model selection, and goodness-of-fit.  Cross-validation tools are also
>>> +available for measuring the accuracy of ABC estimates, and to calculate the
>>> +misclassification probabilities of different models.")
>> Please use full sentences.  For CRAN packages the official description
>> usually provides only a sentence fragment in place of the first
>> sentence.  Please also use “@dfn{Approximate Bayesian Computation}
>> (ABC)” the first time “ABC” is mentioned.
>>
>> All thoughout the indentation is a bit off (that’s expected as the
>> importer is very simple and doesn’t know about the expected
>> indentation).  If you’re using Emacs you can easily adjust it with M-C-q
>> (also known as “indent-sexp”); otherwise please use
>> “etc/indent-code.el”.
>>
>> Have you built this package with Guix?
>>
>> Finally, please rebase this patch so that it can be applied
>> independently from the other patch you’ve sent, which cannot currently
>> be applied due to problems in building the defined package.  Rebasing
>> this patch ensures that it can be merged even if the other patch is
>> stuck.
>>
>> Thanks!
>
>

-- 
Regards
Sahithi

[Message part 2 (text/html, inline)]

Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Thu, 29 Mar 2018 10:34:01 GMT) Full text and rfc822 format available.

Message #19 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Sahithi Yarlagadda <sahi <at> swecha.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Thu, 29 Mar 2018 12:32:25 +0200
Hi Sahithi,

> 1) Cloning the Source
>
> 2) Running guix environment guix
>
> 3) ./bootstrap
>
> 4) ./configure --localstatedir=/var

You need to perform these steps only once.

> 5) make
>
> 6) Make Changes to the cran.scm file
>
> 7) Build using ./pre-inst-env guix build <package>
>
> 8) If build is success, format the patch and submit the patch

I suggest running make again after making changes to any file in the
Guix source tree.  This way you can avoid the “.go file is older than
the .scm” messages.

> *When i am planning to write a different package description, i am
> following this*
>
> 9) rebasing the patch
>
> 10) Pull for any changes
>
> 11) running the process from (2)

I’d do this instead:

    git fetch origin  # fetch any upstream changes
    git stash         # hide my uncommitted local changes
    git rebase origin/master # rebase my local commits on top of upstream
    git stash pop     # apply my uncommitted local changes
    make              # compile .scm to .go

Note that you should be inside of a suitable environment when running
“make”, so better do all of this after “guix environment guix”.

--
Ricardo

GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
https://elephly.net






Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Thu, 29 Mar 2018 10:52:01 GMT) Full text and rfc822 format available.

Message #22 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Sahithi Yarlagadda <sahi <at> swecha.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Thu, 29 Mar 2018 16:20:57 +0530
Thanks Ricardo

When i am building r-abc  I found errors with the following.

  (propagated-inputs
     `(("r-abc-data" ,r-abc-data)


abc.data is the upstream name of abc-data. I am figuring out how to use
it. I have searched for various package description to compare the
usage, but in vain. Can u help me what attribute is to be given to make
it work.



On Thursday 29 March 2018 04:02 PM, Ricardo Wurmus wrote:
> Hi Sahithi,
>
>> 1) Cloning the Source
>>
>> 2) Running guix environment guix
>>
>> 3) ./bootstrap
>>
>> 4) ./configure --localstatedir=/var
> You need to perform these steps only once.
>
>> 5) make
>>
>> 6) Make Changes to the cran.scm file
>>
>> 7) Build using ./pre-inst-env guix build <package>
>>
>> 8) If build is success, format the patch and submit the patch
> I suggest running make again after making changes to any file in the
> Guix source tree.  This way you can avoid the “.go file is older than
> the .scm” messages.
>
>> *When i am planning to write a different package description, i am
>> following this*
>>
>> 9) rebasing the patch
>>
>> 10) Pull for any changes
>>
>> 11) running the process from (2)
> I’d do this instead:
>
>     git fetch origin  # fetch any upstream changes
>     git stash         # hide my uncommitted local changes
>     git rebase origin/master # rebase my local commits on top of upstream
>     git stash pop     # apply my uncommitted local changes
>     make              # compile .scm to .go
>
> Note that you should be inside of a suitable environment when running
> “make”, so better do all of this after “guix environment guix”.
>
> --
> Ricardo
>
> GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
> https://elephly.net
>
>
>

-- 
Regards
Sahithi





Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Thu, 29 Mar 2018 11:01:01 GMT) Full text and rfc822 format available.

Message #25 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Sahithi Yarlagadda <sahi <at> swecha.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Thu, 29 Mar 2018 16:30:48 +0530
Hi

I am planning to do this.

I will submit the Package Description for r-abc-data. Which builds and
create a derivative in gnu/store. Later i will send the abc package
description, where the build finds necessary data and does a successful
build.


On Thursday 29 March 2018 04:20 PM, Sahithi Yarlagadda wrote:
> Thanks Ricardo
>
> When i am building r-abc  I found errors with the following.
>
>   (propagated-inputs
>      `(("r-abc-data" ,r-abc-data)
>
>
> abc.data is the upstream name of abc-data. I am figuring out how to use
> it. I have searched for various package description to compare the
> usage, but in vain. Can u help me what attribute is to be given to make
> it work.
>
>
>
> On Thursday 29 March 2018 04:02 PM, Ricardo Wurmus wrote:
>> Hi Sahithi,
>>
>>> 1) Cloning the Source
>>>
>>> 2) Running guix environment guix
>>>
>>> 3) ./bootstrap
>>>
>>> 4) ./configure --localstatedir=/var
>> You need to perform these steps only once.
>>
>>> 5) make
>>>
>>> 6) Make Changes to the cran.scm file
>>>
>>> 7) Build using ./pre-inst-env guix build <package>
>>>
>>> 8) If build is success, format the patch and submit the patch
>> I suggest running make again after making changes to any file in the
>> Guix source tree.  This way you can avoid the “.go file is older than
>> the .scm” messages.
>>
>>> *When i am planning to write a different package description, i am
>>> following this*
>>>
>>> 9) rebasing the patch
>>>
>>> 10) Pull for any changes
>>>
>>> 11) running the process from (2)
>> I’d do this instead:
>>
>>     git fetch origin  # fetch any upstream changes
>>     git stash         # hide my uncommitted local changes
>>     git rebase origin/master # rebase my local commits on top of upstream
>>     git stash pop     # apply my uncommitted local changes
>>     make              # compile .scm to .go
>>
>> Note that you should be inside of a suitable environment when running
>> “make”, so better do all of this after “guix environment guix”.
>>
>> --
>> Ricardo
>>
>> GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
>> https://elephly.net
>>
>>
>>

-- 
Regards
Sahithi





Information forwarded to guix-patches <at> gnu.org:
bug#30894; Package guix-patches. (Thu, 29 Mar 2018 11:30:02 GMT) Full text and rfc822 format available.

Message #28 received at 30894 <at> debbugs.gnu.org (full text, mbox):

From: Sahithi Yarlagadda <sahi <at> swecha.net>
To: Ricardo Wurmus <rekado <at> elephly.net>
Cc: 30894 <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Thu, 29 Mar 2018 16:58:59 +0530
[Message part 1 (text/plain, inline)]
Hi

On Thursday 29 March 2018 04:30 PM, Sahithi Yarlagadda wrote:
> Hi
>
> I am planning to do this.
>
> I will submit the Package Description for r-abc-data. Which builds and
> create a derivative in gnu/store. Later i will send the abc package
> description, where the build finds necessary data and does a successful
> build.

Submitted a Patch file for r-abc-data which is a dependency for r-abc

http://debbugs.gnu.org/cgi/bugreport.cgi?bug=30983


Then,
Re-based the source

    git fetch origin  # fetch any upstream changes
    git stash         # hide my uncommitted local changes
    git rebase origin/master # rebase my local commits on top of upstream
    git stash pop     # apply my uncommitted local changes
    make              # compile .scm to .go


I have done adding r-abc. And the build was success.

 ( I am still unsure whether i did it right. Fingers Crossed !!!! ...
Waiting for your comment)

Its a great experience while learning with each contribution. Each
Package Description have a different things to look into carefully and
contribute.

Ricardo have been of great help in this process.

Created a Patch (I sense something's not right here!!!). 
Submitting for the review.



>
> On Thursday 29 March 2018 04:20 PM, Sahithi Yarlagadda wrote:
>> Thanks Ricardo
>>
>> When i am building r-abc  I found errors with the following.
>>
>>   (propagated-inputs
>>      `(("r-abc-data" ,r-abc-data)
>>
>>
>> abc.data is the upstream name of abc-data. I am figuring out how to use
>> it. I have searched for various package description to compare the
>> usage, but in vain. Can u help me what attribute is to be given to make
>> it work.
>>
>>
>>
>> On Thursday 29 March 2018 04:02 PM, Ricardo Wurmus wrote:
>>> Hi Sahithi,
>>>
>>>> 1) Cloning the Source
>>>>
>>>> 2) Running guix environment guix
>>>>
>>>> 3) ./bootstrap
>>>>
>>>> 4) ./configure --localstatedir=/var
>>> You need to perform these steps only once.
>>>
>>>> 5) make
>>>>
>>>> 6) Make Changes to the cran.scm file
>>>>
>>>> 7) Build using ./pre-inst-env guix build <package>
>>>>
>>>> 8) If build is success, format the patch and submit the patch
>>> I suggest running make again after making changes to any file in the
>>> Guix source tree.  This way you can avoid the “.go file is older than
>>> the .scm” messages.
>>>
>>>> *When i am planning to write a different package description, i am
>>>> following this*
>>>>
>>>> 9) rebasing the patch
>>>>
>>>> 10) Pull for any changes
>>>>
>>>> 11) running the process from (2)
>>> I’d do this instead:
>>>
>>>     git fetch origin  # fetch any upstream changes
>>>     git stash         # hide my uncommitted local changes
>>>     git rebase origin/master # rebase my local commits on top of upstream
>>>     git stash pop     # apply my uncommitted local changes
>>>     make              # compile .scm to .go
>>>
>>> Note that you should be inside of a suitable environment when running
>>> “make”, so better do all of this after “guix environment guix”.
>>>
>>> --
>>> Ricardo
>>>
>>> GPG: BCA6 89B6 3655 3801 C3C6  2150 197A 5888 235F ACAC
>>> https://elephly.net
>>>
>>>
>>>

-- 
Regards
Sahithi

[0001-gnu-Add-r-abc.patch (text/x-patch, attachment)]

Reply sent to Ricardo Wurmus <rekado <at> elephly.net>:
You have taken responsibility. (Mon, 02 Apr 2018 21:28:01 GMT) Full text and rfc822 format available.

Notification sent to Sahithi Yarlagadda <sahi <at> swecha.net>:
bug acknowledged by developer. (Mon, 02 Apr 2018 21:28:02 GMT) Full text and rfc822 format available.

Message #33 received at 30894-done <at> debbugs.gnu.org (full text, mbox):

From: Ricardo Wurmus <rekado <at> elephly.net>
To: Sahithi Yarlagadda <sahi <at> swecha.net>
Cc: 30894-done <at> debbugs.gnu.org
Subject: Re: [bug#30894] Submission of patch for Add r-abc
Date: Mon, 02 Apr 2018 23:26:11 +0200
Hi Sahithi,

> I have done adding r-abc. And the build was success.

Excellent!  I also built it without problems.

I couldn’t apply it cleanly after making modifications to r-abc-data,
but that’s not your problem.  I performed these minor changes:

* changed license from gpl3 to gpl3+
* added a beginning to the description.

Pushed the patch to the master branch with commit 82c8e0ae0.

Thanks again!

~~ Ricardo






bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Tue, 01 May 2018 11:24:05 GMT) Full text and rfc822 format available.

This bug report was last modified 5 years and 333 days ago.

Previous Next


GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.