Bug 271682 - devel/kommit: update WWW and description
Summary: devel/kommit: update WWW and description
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Some People
Assignee: Joel Bodenmann
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2023-05-27 18:35 UTC by Graham Perrin
Modified: 2023-10-19 15:33 UTC (History)
5 users (show)

See Also:
tcberner: maintainer-feedback+


Attachments
devel/kommit: update WWW and description (1.10 KB, patch)
2023-08-05 00:27 UTC, Andrey Korobkov
no flags Details | Diff
devel/kommit: update WWW and description (fixed) (1.10 KB, patch)
2023-08-05 00:50 UTC, Andrey Korobkov
no flags Details | Diff
[PATCH] devel/kommit: Update pkg-descr and WWW (1.11 KB, patch)
2023-08-05 10:06 UTC, Andrey Korobkov
no flags Details | Diff
[PATCH] devel/kommit: Update pkg-descr and WWW (1.41 KB, patch)
2023-08-06 13:44 UTC, Andrey Korobkov
no flags Details | Diff
[PATCH] devel/kommit: Update pkg-descr and WWW (1.10 KB, patch)
2023-10-15 20:56 UTC, Andrey Korobkov
alster: maintainer-approval? (kde)
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Graham Perrin freebsd_committer freebsd_triage 2023-05-27 18:35:57 UTC
<https://github.com/HamedMasafi/GitKlient#readme>: 

> The project was transferred to KDE. Visit the following address to view 
> the source.
> 
> https://invent.kde.org/sdk/gitklient


https://invent.kde.org/sdk/kommit

> Project 'sdk/gitklient' was moved to 'sdk/kommit'. Please update any 
> links and bookmarks that may still have the old path.


In the description: uppercase G for Git. 

Thanks
Comment 1 Andrey Korobkov 2023-08-05 00:27:43 UTC
Created attachment 243850 [details]
devel/kommit: update WWW and description
Comment 2 Andrey Korobkov 2023-08-05 00:50:52 UTC
Created attachment 243851 [details]
devel/kommit: update WWW and description (fixed)
Comment 3 Andrey Korobkov 2023-08-05 10:06:55 UTC
Created attachment 243857 [details]
[PATCH] devel/kommit: Update pkg-descr and WWW
Comment 4 Raphael Kubo da Costa freebsd_committer freebsd_triage 2023-08-05 17:21:07 UTC
Comment on attachment 243857 [details]
[PATCH] devel/kommit: Update pkg-descr and WWW

Thanks for the patch. A few notes:

* I don't think you need to bump PORTREVISION since you're basically only editing some metadata.
* Doesn't it make more sense to point WWW to https://apps.kde.org/kommit/ (the app page) instead of the git repository?
Comment 5 Graham Perrin 2023-08-05 21:28:18 UTC
(In reply to Raphael Kubo da Costa from comment #4)

> … https://apps.kde.org/kommit/

That's better, I wasn't aware of this page. Thanks.
Comment 6 Andrey Korobkov 2023-08-06 13:44:07 UTC
Created attachment 243887 [details]
[PATCH] devel/kommit: Update pkg-descr and WWW
Comment 7 Andrey Korobkov 2023-10-15 20:56:30 UTC
Created attachment 245649 [details]
[PATCH] devel/kommit: Update pkg-descr and WWW
Comment 8 Tobias C. Berner freebsd_committer freebsd_triage 2023-10-16 05:00:59 UTC
lgtm
Comment 9 commit-hook freebsd_committer freebsd_triage 2023-10-16 16:21:18 UTC
A commit in branch main references this bug:

URL: https://cgit.FreeBSD.org/ports/commit/?id=fb5043d6ed2cdbfe4fa8555178aca9184cc6a9a0

commit fb5043d6ed2cdbfe4fa8555178aca9184cc6a9a0
Author:     Graham Perrin <grahamperrin@gmail.com>
AuthorDate: 2023-10-16 16:17:01 +0000
Commit:     Joel Bodenmann <jbo@FreeBSD.org>
CommitDate: 2023-10-16 16:19:09 +0000

    devel/kommit: Update WWW and description

    PR:             271682
    Approved by:    tcberner (mentor)

 devel/kommit/Makefile  | 6 +++---
 devel/kommit/pkg-descr | 2 +-
 2 files changed, 4 insertions(+), 4 deletions(-)
Comment 10 Joel Bodenmann freebsd_committer freebsd_triage 2023-10-16 16:21:33 UTC
Committed - Thanks!
Comment 11 Andrey Korobkov 2023-10-16 17:34:22 UTC
Thanks for reviewing and committing my patch.
It may look somewhat childish from me, as I like my authorship to be stated in commit log :)
Actually I'm not a bit jealous to respective Graham Perrin, I just like to have more my patches being accepted :)
Could you, please, fix it?
Comment 12 Tobias C. Berner freebsd_committer freebsd_triage 2023-10-17 06:00:15 UTC
(In reply to Andrey Korobkov from comment #11)

Moin moin 

Oh, this was an unfortunate misunderstanding. There is no repo-surgery however, so we cannot amend it. Thanks a lot for your contribution, and we'll make sure to do the proper attribution next time.

(Btw, if you upload a git-format patch (e.g. running 'git format-patch -1'), then author information is automatically correct :) )

mfg Tobias
Comment 13 Graham Perrin 2023-10-17 07:26:35 UTC
Hi

(In reply to Tobias C. Berner from comment #12)

Was attachment 245649 [details] not correct? 

Here, with an outdated copy of the tree: 

% git -C /usr/ports am /tmp/0001-devel-kommit-Update-pkg-descr-and-WWW.patch
Applying: devel/kommit: Update pkg-descr and WWW
% git -C /usr/ports log -n 1
commit 4e223bdb1015ba22daecfdba609c0371bdc5bdd6 (HEAD -> main)
Author: Andrey Korobkov <alster@vinterdalen.se>
Date:   Sat Aug 5 04:23:17 2023 +0400

    devel/kommit: Update pkg-descr and WWW
    
    PR: 271682
% 


git-am(1)
<https://git-scm.com/docs/git-am>
<https://man.freebsd.org/cgi/man.cgi?query=git-am&sektion=1&manpath=freebsd-ports>
Comment 14 Graham Perrin 2023-10-17 08:18:28 UTC
> Hi

… I forgot to say, thanks!
Comment 15 Andrey Korobkov 2023-10-17 21:27:35 UTC
Hi and thanks for peacefully resolving the issue in kind and polite way! :)
Let's have it closed.
I like to work alongside with all of you on such an interesting project as FreeBSD and already work on future patches.

Best regards,
Andrey
Comment 16 Joel Bodenmann freebsd_committer freebsd_triage 2023-10-17 21:30:55 UTC
Andrey, apologies for the mistake - entirely my fault. This was not intentional at all.
Comment 17 Andrey Korobkov 2023-10-18 00:07:12 UTC
No problem at all, Joel, don't worry!

I'm even glad this small mistake happened as it showed me how friendly and polite people are in this community. :)

I also would like to became a committer one day.
And now I'm sure you all will also be kind and supportive, would I make any mistake.

Thank you!
Comment 18 Graham Perrin 2023-10-19 15:33:21 UTC
(In reply to Tobias C. Berner from comment #12)

tcberner@

My apologies: when I posted <https://lists.freebsd.org/archives/dev-commits-ports-all/2023-October/084667.html>, in the list of three CC recipients I misspelt the domain alongside your user ID, so you probably did not notice the post.

(I had terrible trouble, at the time, with the port of Thunderbird, problems far broader than the description at <https://bugzilla.mozilla.org/show_bug.cgi?id=1856513>. It wasn't responding properly to keystrokes, I should have paid closer attention before sending the email. Sorry for the carelessness.)