unofficial mirror of guix-devel@gnu.org 
 help / color / mirror / code / Atom feed
From: "Gábor Boskovits" <boskovits@gmail.com>
To: Jan Wielkiewicz <tona_kosmicznego_smiecia@interia.pl>
Cc: Guix-devel <guix-devel@gnu.org>
Subject: Re: Maintaining GNU Jami package for Guix
Date: Mon, 4 Nov 2019 11:02:59 +0100	[thread overview]
Message-ID: <CAE4v=ph=dQ=wRmqjp+LQBD+x0eNgBfuutn-LHzLrMWaR1=CeXw@mail.gmail.com> (raw)
In-Reply-To: <20191103115420.6221e83b@interia.pl>

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

Hello Jan,

Jan Wielkiewicz <tona_kosmicznego_smiecia@interia.pl> ezt írta (időpont:
2019. nov. 3., V, 22:20):

> Dnia 2019-11-03, o godz. 11:15:56
> Pierre Neidhardt <mail@ambrevar.xyz> napisał(a):
>
> > Can you share the complete recipe?
> >
>
> Yes. I guess you want a patch of my entire work so far, because
> pjproject-jami is meaningless without changes in jami, so I'm attaching
> the whole patch. Also the commit messages are messy, any advice how
> should they look?
>
>
Thanks for working on this. I can share some recommendations on the commit
messages and structuring
changes. While developing, feel free to do however it is convenient for you.
Then before sumitting:
If I see it correctly, then three packages are involved, so this should be
three commits.
When adding a new package make that a separate commit.
Make it one commit, the final version. (You can squash commits together to
make it one.)
The commit message there should be in the form:
gnu: Add <package>.
 When updating a package, you can make that a single commit.
Commit message should be:
gnu: <package>: Update to <version>.

If anything else is done besides updating the version, mention that in the
commit message.
You can find examples in the git log of guix, like for commit:
af561664b8191e2a34570176422f4d500d5fd4fb

One further thing: If you order the patches, then the earlier patches can
make it to upstream before the rest of the series.
For example libresample could be incorporated eariler, reducing the amount
of code on the branch.

Thanks so much for working on Jami, and happy hacking!


> Jan Wielkiewicz
>

Best regards,
g_bor
-- 
OpenPGP Key Fingerprint: 7988:3B9F:7D6A:4DBF:3719:0367:2506:A96C:CF63:0B21

[-- Attachment #2: Type: text/html, Size: 2668 bytes --]

  reply	other threads:[~2019-11-04 10:03 UTC|newest]

Thread overview: 40+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2019-10-18 18:56 Maintaining GNU Jami package for Guix Jan
2019-10-18 19:28 ` Marius Bakke
2019-10-19  9:02   ` Pierre Neidhardt
2019-10-19 20:38     ` Jan Wielkiewicz
2019-10-19 20:52       ` Ricardo Wurmus
2019-10-19 22:37         ` Jan
2019-10-20 11:14           ` Ricardo Wurmus
2019-10-20 11:51           ` Pierre Neidhardt
2019-10-20 11:48       ` Pierre Neidhardt
2019-10-25 15:03         ` Pierre Neidhardt
2019-10-25 19:44           ` Jan Wielkiewicz
2019-10-26 10:12             ` Pierre Neidhardt
2019-10-26 23:24               ` Jan Wielkiewicz
2019-10-28  7:53                 ` Pierre Neidhardt
2019-10-27 18:13 ` Jan Wielkiewicz
2019-10-27 18:18   ` Pierre Neidhardt
2019-10-27 19:23     ` Jan Wielkiewicz
2019-10-27 21:52 ` Jan Wielkiewicz
2019-10-31 20:19 ` Jan Wielkiewicz
2019-10-31 21:37   ` Pierre Neidhardt
2019-10-31 22:26   ` Marius Bakke
2019-11-01 12:01     ` Jan Wielkiewicz
2019-11-01 19:02       ` Pierre Neidhardt
2019-11-01 20:42         ` Jan Wielkiewicz
2019-11-02  9:38           ` Pierre Neidhardt
2019-11-03 16:37       ` Marius Bakke
2019-11-03 17:43         ` Jan
2019-11-03 18:06           ` Marius Bakke
2019-11-01 12:29 ` Jan Wielkiewicz
2019-11-01 19:01   ` Pierre Neidhardt
2019-11-01 23:16     ` Jan Wielkiewicz
2019-11-03 10:15       ` Pierre Neidhardt
2019-11-03 10:54         ` Jan Wielkiewicz
2019-11-04 10:02           ` Gábor Boskovits [this message]
2019-11-03 10:59         ` Jan Wielkiewicz
2019-11-03 17:07   ` Marius Bakke
2019-11-03 17:47     ` Jan
2019-11-03 18:12   ` Marius Bakke
2019-11-03 18:27     ` Jan
2019-11-03 18:52       ` Marius Bakke

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://guix.gnu.org/

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

  git send-email \
    --in-reply-to='CAE4v=ph=dQ=wRmqjp+LQBD+x0eNgBfuutn-LHzLrMWaR1=CeXw@mail.gmail.com' \
    --to=boskovits@gmail.com \
    --cc=guix-devel@gnu.org \
    --cc=tona_kosmicznego_smiecia@interia.pl \
    /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 public inbox

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

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).