* [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl
@ 2022-02-16 1:49 J. Dekker
2022-02-16 1:49 ` [FFmpeg-devel] [PATCH 2/2] mailmap: update entry J. Dekker
2022-02-24 13:18 ` [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl Thilo Borgmann
0 siblings, 2 replies; 6+ messages in thread
From: J. Dekker @ 2022-02-16 1:49 UTC (permalink / raw)
To: ffmpeg-devel
This script generates the current general assembly voters according to
the criteria of '20 commits in the last 36 months'.
Signed-off-by: J. Dekker <jdek@itanimul.li>
---
| 3 +++
tools/general_assembly.pl | 40 ++++++++++++++++++++++++++++++++++
2 files changed, 43 insertions(+)
create mode 100644 tools/general_assembly.pl
--git a/doc/dev_community/community.md b/doc/dev_community/community.md
index 21e08e20e3..516ca5c05e 100644
--- a/doc/dev_community/community.md
+++ b/doc/dev_community/community.md
@@ -25,6 +25,9 @@ proposal by a member of the General Assembly.
They are part of the GA for two years, after which they need a confirmation by
the GA.
+A script to generate the current members of the general assembly (minus members
+voted in) can be found in `tools/general_assembly.pl`.
+
## Voting
Voting is done using a ranked voting system, currently running on https://vote.ffmpeg.org/ .
diff --git a/tools/general_assembly.pl b/tools/general_assembly.pl
new file mode 100644
index 0000000000..898a6262ef
--- /dev/null
+++ b/tools/general_assembly.pl
@@ -0,0 +1,40 @@
+#!/usr/bin/env perl
+
+use warnings;
+use strict;
+
+use POSIX qw(strftime);
+use Encode qw(decode);
+use Data::Dumper;
+
+sub trim { my $s = shift; $s =~ s/^\s+|\s+$//g; return $s };
+
+my @shortlog = split /\n/, decode('UTF-8', `git log --pretty=format:"%aN <%aE>" --since="last 36 months" | sort | uniq -c | sort -r`, Encode::FB_CROAK);
+my %assembly = ();
+
+foreach my $line (@shortlog) {
+ my ($count, $name, $email) = $line =~ m/^ *(\d+) *(.*?) <(.*?)>/;
+ if ($count < 20) {
+ next;
+ }
+
+ $name = trim $name;
+ if ($count < 50) {
+ my $true = 0;
+ my @commits = split /(^|\n)commit [a-z0-9]{40}(\n|$)/, decode('UTF-8', `git log --name-only --use-mailmap --author="$email" --since="last 36 months"`, Encode::FB_CROAK);
+ foreach my $commit (@commits) {
+ $true++; # if ($commit =~ /\n[\w\/]+\.(c|h|S|asm|texi)/);
+ }
+
+ if ($true < 20) {
+ next;
+ }
+ }
+
+ $assembly{$name} = $email;
+}
+
+printf("# %s %s", strftime("%Y-%m-%d", localtime), decode('UTF-8', `git rev-parse HEAD`, Encode::FB_CROAK));
+foreach my $email (sort values %assembly) {
+ printf("%s\n", $email);
+}
--
2.32.0 (Apple Git-132)
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* [FFmpeg-devel] [PATCH 2/2] mailmap: update entry
2022-02-16 1:49 [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl J. Dekker
@ 2022-02-16 1:49 ` J. Dekker
2022-02-24 13:18 ` [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl Thilo Borgmann
1 sibling, 0 replies; 6+ messages in thread
From: J. Dekker @ 2022-02-16 1:49 UTC (permalink / raw)
To: ffmpeg-devel
Signed-off-by: J. Dekker <jdek@itanimul.li>
---
.mailmap | 3 ++-
1 file changed, 2 insertions(+), 1 deletion(-)
diff --git a/.mailmap b/.mailmap
index ba072f38c8..5544fc5b5c 100644
--- a/.mailmap
+++ b/.mailmap
@@ -8,7 +8,8 @@
<stebbins@jetheaddev.com> <jstebbins@jetheaddev.com>
<barryjzhao@tencent.com> <mypopydev@gmail.com>
<barryjzhao@tencent.com> <jun.zhao@intel.com>
-<josh@itanimul.li> <joshdk@obe.tv>
+<jdek@itanimul.li> <joshdk@obe.tv>
+J. Dekker <jdek@itanimul.li> <josh@itanimul.li>
<michael@niedermayer.cc> <michaelni@gmx.at>
<linjie.justin.fu@gmail.com> <linjie.fu@intel.com>
<linjie.justin.fu@gmail.com> <fulinjie@zju.edu.cn>
--
2.32.0 (Apple Git-132)
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl
2022-02-16 1:49 [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl J. Dekker
2022-02-16 1:49 ` [FFmpeg-devel] [PATCH 2/2] mailmap: update entry J. Dekker
@ 2022-02-24 13:18 ` Thilo Borgmann
2022-02-24 13:46 ` Nicolas George
1 sibling, 1 reply; 6+ messages in thread
From: Thilo Borgmann @ 2022-02-24 13:18 UTC (permalink / raw)
To: ffmpeg-devel
Am 16.02.22 um 02:49 schrieb J. Dekker:
> This script generates the current general assembly voters according to
> the criteria of '20 commits in the last 36 months'.
>
> Signed-off-by: J. Dekker <jdek@itanimul.li>
> ---
> doc/dev_community/community.md | 3 +++
> tools/general_assembly.pl | 40 ++++++++++++++++++++++++++++++++++
> 2 files changed, 43 insertions(+)
> create mode 100644 tools/general_assembly.pl
Both LGTM.
-Thilo
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl
2022-02-24 13:18 ` [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl Thilo Borgmann
@ 2022-02-24 13:46 ` Nicolas George
2022-03-01 11:17 ` J. Dekker
0 siblings, 1 reply; 6+ messages in thread
From: Nicolas George @ 2022-02-24 13:46 UTC (permalink / raw)
To: FFmpeg development discussions and patches
[-- Attachment #1.1: Type: text/plain, Size: 546 bytes --]
Thilo Borgman (12022-02-24):
> Both LGTM.
Does it?
The way I remember it, this condition was for the initial assembly, as a
temporary measure. I might have missed some steps, but I do not remember
we adopted this as a constant rule.
I see several flaws with it:
- Documentation patches were not counted, now they are.
- Cosmetic patches are counted.
- It does not take the size of the changes.
Using an imperfect solution for bootstrap is one thing, using it
permanently is another.
Regards,
--
Nicolas George
[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
[-- Attachment #2: Type: text/plain, Size: 251 bytes --]
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl
2022-02-24 13:46 ` Nicolas George
@ 2022-03-01 11:17 ` J. Dekker
2023-02-08 15:50 ` Anton Khirnov
0 siblings, 1 reply; 6+ messages in thread
From: J. Dekker @ 2022-03-01 11:17 UTC (permalink / raw)
To: FFmpeg development discussions and patches
On 24 Feb 2022, at 14:46, Nicolas George wrote:
> Thilo Borgman (12022-02-24):
>> Both LGTM.
>
> Does it?
An earlier reply would have been helpful here, i.e. before an ACK rather than after.
> The way I remember it, this condition was for the initial assembly, as a
> temporary measure. I might have missed some steps, but I do not remember
> we adopted this as a constant rule.
>
> I see several flaws with it:
>
> - Documentation patches were not counted, now they are.
>
> - Cosmetic patches are counted.
>
> - It does not take the size of the changes.
>
> Using an imperfect solution for bootstrap is one thing, using it
> permanently is another.
>
It is only supposed to match the documentation. Of course, nothing says it can't be changed later when the 'bootstrap' process is completed (and let's consider that the last time this was actually discussed was in Tokyo, not sure what good delaying it further helps).
You are welcome to send a patch to our documentation to clarify the rules if you think they are not correct. However, I personally think that any contribution to the project is useful and documentation & cosmetic improvements should not be penalized needlessly.
--
J. Dekker
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
* Re: [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl
2022-03-01 11:17 ` J. Dekker
@ 2023-02-08 15:50 ` Anton Khirnov
0 siblings, 0 replies; 6+ messages in thread
From: Anton Khirnov @ 2023-02-08 15:50 UTC (permalink / raw)
To: FFmpeg development discussions and patches
Since Nicolas did not follow up, shall we go ahead and push this?
Something so simple really should not be stuck in place for a year.
--
Anton Khirnov
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
To unsubscribe, visit link above, or email
ffmpeg-devel-request@ffmpeg.org with subject "unsubscribe".
^ permalink raw reply [flat|nested] 6+ messages in thread
end of thread, other threads:[~2023-02-08 15:50 UTC | newest]
Thread overview: 6+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2022-02-16 1:49 [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl J. Dekker
2022-02-16 1:49 ` [FFmpeg-devel] [PATCH 2/2] mailmap: update entry J. Dekker
2022-02-24 13:18 ` [FFmpeg-devel] [PATCH v2 1/2] tools: add general_assembly.pl Thilo Borgmann
2022-02-24 13:46 ` Nicolas George
2022-03-01 11:17 ` J. Dekker
2023-02-08 15:50 ` Anton Khirnov
Git Inbox Mirror of the ffmpeg-devel mailing list - see https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
This inbox may be cloned and mirrored by anyone:
git clone --mirror https://master.gitmailbox.com/ffmpegdev/0 ffmpegdev/git/0.git
# If you have public-inbox 1.1+ installed, you may
# initialize and index your mirror using the following commands:
public-inbox-init -V2 ffmpegdev ffmpegdev/ https://master.gitmailbox.com/ffmpegdev \
ffmpegdev@gitmailbox.com
public-inbox-index ffmpegdev
Example config snippet for mirrors.
AGPL code for this site: git clone https://public-inbox.org/public-inbox.git