unofficial mirror of guile-user@gnu.org 
 help / color / mirror / Atom feed
From: Zelphir Kaltstahl <zelphirkaltstahl@posteo.de>
To: alex.sassmannshausen@gmail.com
Cc: guile-user@gnu.org
Subject: Re: guile-hall issues converting my project to a hall project
Date: Thu, 18 Feb 2021 20:16:42 +0100	[thread overview]
Message-ID: <dbfeff36-5cf4-0c70-4873-833fc03c44dd@posteo.de> (raw)
In-Reply-To: <87wnv6xulu.fsf@gmail.com>

Hi Alex!

I am willing to test/try some more.

When you say "since commit xyz", do you mean commit of guile-hall
itself, or a commit of Guix, where an updated version of guile-hall is
available?

How would I test the master branch version? I would guess install it in
a guix environment and then use that environment to try and convert a
project.

What would you suggest testing?

It was quite OK to edit hall.scm in an editor. Perhaps, if a project is
significantly bigger than my project, it would become cumbersome, but
for me personally it is fine, now that I know what goes in there and how
it needs to look.

I have not yet begun writing a guide for converting a project. Hopefully
I'll be able to do so soon.

Best regards,
Zelphir

On 2/17/21 10:17 PM, Alex Sassmannshausen wrote:
> Hi Zelphir and Tim,
>
> I am the author of guile hall — apologies for only now getting into this
> thread. I'm afraid I have been somewhat distracted with other things.
>
> First of all I want to echo what others have said — thank you very much
> for your detailed descriptions of what exactly happened when you tried
> to migrate the project to guile hall.
>
> The aim of the project is to massively reduce the barrier of entry to
> creating new, portable, high quality guile projects — and to contribute
> them to Guix. Your descriptions suggest it's not there yet!
>
> In any case, some comments inline:
>
> Zelphir Kaltstahl <zelphirkaltstahl@posteo.de> writes:
>
>> Hello Tim!
>>
>> Thank you, it works now!
>>
>> Removing the duplicate entry of `fslib` in hall.scm fixed it.
> Fwiw, since commit ac76541a this issue can be automatically resolved by
> running scan once more: it should remove duplicate entries in hall.scm.
>
>> Regarding the license: OK, I have no problem moving my license to
>> `COPYING`. However, I still think, that it should not put GPL there,
>> when I specified AGPL in `hall.scm`. This looks like a hardcoded
>> fallback, which does not take the license specification into account.
>> Something like: "If there is no `COPYING` file just put GPL into a file
>> `COPYING`." instead of "If there is no `COPYING` file just put
>> <user-specified-license-in-hall.scm> into a file `COPYING`.". I could be
>> wrong though, as I do not know anything about guile-hall's internals.
> This is odd — Hall should respect your license choice. It should, for
> (A)GPL licenses automatically download those from the internet and
> install them in COPYING. There was an issue with those licenses hiding
> behind a 302 status code, which resulted in fallback text being loaded
> in COPYING. But even that fallback text should respect your license.
>
> I just tried changing my license to AGPLv3+ in one of my projects,
> running hall from Master (the most recent commit fixes the 302 license
> issue), and it's fetching the license correctly.
>
> I'd be interested in seeing what your experience is if you are willing
> to try?
>
> In any case, Hall is undergoing active development, and I'm hoping to
> have a fresh release end of this month, with a whole bunch of
> improvements and bug fixes. Definitely feel free to drop issues to me
> directly by email or on gitlab (though some of the ones you raised have
> been fixed, like the unknown filetype issue).
>
> Best wishes,
>
> Alex
>
>> Follow up question would be, how to bring the package into the guix
>> repository, but I am guessing, that it will be answered at
>> https://guix.gnu.org/cookbook/en/html_node/Direct-checkout-hacking.html,
>> which I have not read yet.
>>
>> Another question is, whether I should put you into the authors file and
>> write something like "help with packaging" there. What is the common
>> practice?
>>
>> Best regards,
>> Zelphir
>>
>> On 2/16/21 5:48 PM, Tim Van den Langenbergh wrote:
>>> Err, looking at your hall.scm file, you have the fslib file added to your
>>> libraries twice.
>>>
>>> Guix environment is not needed if you have all the requirements for building
>>> the package installed locally, but if you want to distribute your package it's
>>> good practise to ensure it builds in a clean environment (see also https://
>>> guix.gnu.org/manual/en/html_node/Invoking-guix-environment.html for more
>>> information about Guix environments).
>>>
>>> The "COPYING" file is hardcoded as license file in Hall, to ensure compatibility
>>> with GNU standards: https://www.gnu.org/licenses/gpl-howto.en.html
>>>
>>> Hope this helps,
>>>
>>> Vale
>>>
>>> -Tim
>>>
>>>
-- 
repositories: https://notabug.org/ZelphirKaltstahl




  reply	other threads:[~2021-02-18 19:16 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2021-02-06 21:49 guile-hall issues converting my project to a hall project Zelphir Kaltstahl
2021-02-07  9:14 ` tomas
2021-02-07 13:30 ` Tim Van den Langenbergh
2021-02-11 20:30   ` Zelphir Kaltstahl
2021-02-16  0:23   ` Zelphir Kaltstahl
2021-02-17 22:47     ` Vladimir Zhbanov
2021-02-18 12:18       ` alex sassmannshausen
     [not found] ` <11206998.46ALo4VoAQ@terra>
     [not found]   ` <02902ff9-1585-a453-4e36-c9a731eee6fe@posteo.de>
2021-02-16 16:48     ` Tim Van den Langenbergh
2021-02-16 18:12       ` Zelphir Kaltstahl
2021-02-17 21:17         ` Alex Sassmannshausen
2021-02-18 19:16           ` Zelphir Kaltstahl [this message]
2021-02-21 13:42             ` Alex Sassmannshausen
2021-02-22 11:34               ` guile-hall packaging guide Zelphir Kaltstahl
2021-02-22 20:52                 ` Jérémy Korwin-Zmijowski
2021-02-23 16:32                   ` Alex Sassmannshausen

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

  List information: https://www.gnu.org/software/guile/

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=dbfeff36-5cf4-0c70-4873-833fc03c44dd@posteo.de \
    --to=zelphirkaltstahl@posteo.de \
    --cc=alex.sassmannshausen@gmail.com \
    --cc=guile-user@gnu.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
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).