all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Tobias Geerinckx-Rice via Guix-patches via <guix-patches@gnu.org>
To: Tim Magee <timothy@eastlincoln.net>
Cc: 42357@debbugs.gnu.org
Subject: [bug#42357] [PATCH] add package definition for Nextcloud Desktop
Date: Wed, 15 Jul 2020 22:29:37 +0200	[thread overview]
Message-ID: <87lfjkpln2.fsf@nckx> (raw)
In-Reply-To: <8ab6db26-8309-750e-b730-2d46a5e6f49a@eastlincoln.net>

[-- Attachment #1: Type: text/plain, Size: 1463 bytes --]

Tim,

Tim Magee 写道:
> What I did is wholesale copy the entire 'sync.scm' file thinking 
> that
> the entire file was necessary for ownCloud in particular. I then 
> copied
> modified the ownCloud definition until Nextcloud would 
> compile. I will
> have to admit that I was a bit confused as to why the ownCloud 
> package
> was in a file called 'sync.scm' but I assumed that there was 
> good reason
> somewhere. Now that I know that multiple packages can be 
> declared in the
> same file, it seems that it would make more sense to place 
> Nextcloud in
> sync.scm as well.
>
> Would placing Nextcloud inside sync.scm be a better idea?

The reason I suggested nextcloud.scm is that there are quite a few 
maintained plug-ins[0] (‘apps’).  It would be nice if those too 
could be installed with Guix.

While that won't happen right away, I prefer having the entire 
stack in a separate file rather than in a ‘NextCloud section’ of 
sync.scm.  It's not of vital importance :-)

I tried to apply your original mail as a patch but it failed.  It 
looks like your MUA wrapped lines where it shouldn't have, 
treating your valuable patch as mungible text.

Try sending the revised version as an attachment, or using git 
send-email[1].

T I A,

T G-R

[0]: https://apps.nextcloud.com/
Disclaimer: click at your own risk, but all the ‘apps’ I checked 
were FSDG-OK.

[1]: guix install git git:send-email

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

  reply	other threads:[~2020-07-15 20:30 UTC|newest]

Thread overview: 7+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
     [not found] <e7ea0a47-a381-6fc9-8885-518e900843d2.ref@eastlincoln.net>
2020-07-14 22:13 ` [bug#42357] [PATCH] add package definition for Nextcloud Desktop Tim Magee
2020-07-14 23:32   ` Tobias Geerinckx-Rice via Guix-patches via
2020-07-14 23:44     ` Tobias Geerinckx-Rice via Guix-patches via
2020-07-15 16:57       ` Tim Magee
2020-07-15 20:29         ` Tobias Geerinckx-Rice via Guix-patches via [this message]
2020-07-17 18:03           ` Jonathan Brielmaier
2023-05-18 15:12   ` bug#42357: " Hartmut Goebel

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

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

  git send-email \
    --in-reply-to=87lfjkpln2.fsf@nckx \
    --to=guix-patches@gnu.org \
    --cc=42357@debbugs.gnu.org \
    --cc=me@tobias.gr \
    --cc=timothy@eastlincoln.net \
    /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.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.