From: Konstantin Kharlamov <Hi-Angel@yandex.ru>
To: "Björn Bidar" <bjorn.bidar@thaodan.de>
Cc: Jim Porter <jporterbugs@gmail.com>,
emacs-devel@gnu.org, Jonas Bernoulli <jonas@bernoul.li>
Subject: Re: Adding git-commit highlight mode?
Date: Sat, 04 Jan 2025 04:45:31 +0300 [thread overview]
Message-ID: <48c9115e57a6abd19b4465ab5d97d15301c395dd.camel@yandex.ru> (raw)
In-Reply-To: <87v7uv9xub.fsf@>
On Sat, 2025-01-04 at 03:22 +0200, Björn Bidar wrote:
> Konstantin Kharlamov <Hi-Angel@yandex.ru> writes:
>
> > On Fri, 2025-01-03 at 23:14 +0200, Björn Bidar wrote:
> > > Jim Porter <jporterbugs@gmail.com> writes:
> > >
> > > > On 1/2/2025 10:30 AM, Konstantin Kharlamov wrote:
> > > > > But Emacs seems to be the only widely popular editor that
> > > > > still
> > > > > doesn't
> > > > > provide OOTB at least syntax highlight for git-commit format.
> > > > > So,
> > > > > does
> > > > > anyone have opposition to adding a major mode that would be
> > > > > bound
> > > > > to
> > > > > filenames like `COMMIT_EDITMSG` and others, and would provide
> > > > > the
> > > > > aforementioned highlight?
> > > >
> > > > For what it's worth, I wrote a very simple package to do this
> > > > for
> > > > myself, since I don't use Magit. (I'm just so used to the Git
> > > > command
> > > > line that I've never taken the time to mess with Magit.)
> > > >
> > >
> > > Is there a way we can do this without reinventing the wheel? E.g.
> > > by
> > > including Jonas's git-commit mode into Emacs?
> >
> > Using Jim's mode comes as close as it gets to not reinventing the
> > wheel. As mentioned by Stefan elsewhere in the thread, Jonas' git-
> > commit that is part of magit can't be easily included for license
> > reasons.
>
> Most of the code was written by Jonas. If either the other authors
> code
> is replaced or they have also assigned their copyright to the FSF it
> could be included.
If you open this link
https://github.com/magit/magit/blame/main/lisp/git-commit.el and look
at the "contributors" text, it says "24". I think contacting 23
different people is so much work that you can rewrite the syntax
highlight 5 times and still get spare time.
> > > PS: It would be very beneficial to not uses Github for Emacs
> > > development but other FOSS platforms such as Codeberg. No need to
> > > feed
> > > Copilot with our code to copy it into other non-FOSS code.
> >
> > As mentioned by Dick, Emacs is mirrored to Github anyway.
>
> Dicks rather trolling comment aside, I don't think that is a reason
> to
> use Github.
>
> > search it seems neither Codeberg nor Gitlab has an active Emacs
> > mirror,
> > right?
> > That means there's no other option besides Github for
> > contributors to store their local changes to before sending them
> > upstream.
>
> I don't see a problem there, create an account on which server you
> prefer and fetch from Savannah.
> No need to use Github to download the Emacs repository.
I guess you're right.
> > Obviously, a new contributor wouldn't have an account to
> > git.savannah, and even if they do, git.savannah doesn't allow to
> > fork a
> > repo, instead it requires to store everyone the changes to their
> > local
> > branch on a shared repo, which seems unsafe on a bigger scale.
>
> This has nothing to do with Savannah, you don't have to "fork" a
> repository that's a Forge thing to map pull requests (and to easier
> map
> commit refs between the source and the fork but that's OT).
Forking is a solution to the problem where you don't want to grant an
arbitrary person write access to a git repo. Though now that I think,
theoretically it may be possible to implement similar workflow by only
granting access to the `username/` sub-tree. Access management is done
by a harness around git, and one can write it any way they want. Either
way, I don't think such access management has been implemented by
anyone, whereas "forking" workflow does have implementations.
next prev parent reply other threads:[~2025-01-04 1:45 UTC|newest]
Thread overview: 37+ messages / expand[flat|nested] mbox.gz Atom feed top
2025-01-02 18:30 Adding git-commit highlight mode? Konstantin Kharlamov
2025-01-02 19:01 ` Eli Zaretskii
2025-01-02 19:07 ` Konstantin Kharlamov
2025-01-02 20:10 ` Stefan Kangas
2025-01-02 21:01 ` Eli Zaretskii
2025-01-02 21:38 ` Stefan Kangas
2025-01-03 5:26 ` Jim Porter
2025-01-02 21:40 ` Konstantin Kharlamov
2025-01-03 5:29 ` Jim Porter
2025-01-03 13:08 ` Jonas Bernoulli
2025-01-02 20:11 ` Eli Zaretskii
2025-01-02 21:19 ` Arsen Arsenović
2025-01-02 21:53 ` Stefan Kangas
2025-01-02 22:27 ` Arsen Arsenović
2025-01-03 21:02 ` Sean Whitton
2025-01-04 7:17 ` Eli Zaretskii
2025-01-04 9:52 ` Sean Whitton
2025-01-15 15:59 ` Konstantin Kharlamov
2025-01-18 18:06 ` Jim Porter
2025-01-03 6:45 ` Eli Zaretskii
2025-01-02 19:17 ` Jim Porter
2025-01-02 19:19 ` Konstantin Kharlamov
2025-01-02 20:17 ` Eli Zaretskii
2025-01-03 21:14 ` Björn Bidar
2025-01-05 14:39 ` Philip Kaludercic
2025-01-05 22:48 ` Konstantin Kharlamov
[not found] ` <87r05jbnw2.fsf@>
2025-01-03 22:52 ` Konstantin Kharlamov
2025-01-04 1:22 ` Björn Bidar
[not found] ` <87v7uv9xub.fsf@>
2025-01-04 1:45 ` Konstantin Kharlamov [this message]
2025-01-05 0:46 ` Björn Bidar
[not found] ` <871pxiulxz.fsf@>
2025-01-05 3:55 ` Konstantin Kharlamov
2025-01-04 12:50 ` Jonas Bernoulli
2025-01-04 17:20 ` Jonas Bernoulli
2025-01-05 0:32 ` Björn Bidar
2025-01-05 18:47 ` Jim Porter
2025-01-05 0:42 ` Björn Bidar
[not found] ` <875xmuum48.fsf@>
2025-01-05 3:49 ` Konstantin Kharlamov
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/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=48c9115e57a6abd19b4465ab5d97d15301c395dd.camel@yandex.ru \
--to=hi-angel@yandex.ru \
--cc=bjorn.bidar@thaodan.de \
--cc=emacs-devel@gnu.org \
--cc=jonas@bernoul.li \
--cc=jporterbugs@gmail.com \
/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/emacs.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).