Re: [FFmpeg-devel] [PATCH 1/2] tools/general_assembly: implement extra GA members

[Michael Niedermayer <michael@niedermayer.cc>]

[-- Attachment #1.1: Type: text/plain, Size: 2574 bytes --]

On Thu, Nov 23, 2023 at 10:14:23AM +0100, Anton Khirnov wrote:
> ---
>  tools/general_assembly.pl | 20 +++++++++++++++++---
>  1 file changed, 17 insertions(+), 3 deletions(-)
> 
> diff --git a/tools/general_assembly.pl b/tools/general_assembly.pl
> index 4c3208ccac..3bf65f3405 100755
> --- a/tools/general_assembly.pl
> +++ b/tools/general_assembly.pl
> @@ -13,6 +13,12 @@ use utf8;
>  use DateTime;
>  use DateTime::Format::ISO8601;
>  
> +my @extra_members = (
> +    # entries should be of the format
> +    # [   <name>,   <email>, <date elected> ],
> +    # ['Foo Bar', 'foo@bar', DateTime->new(year => 8613, month => 5, day => 22)],
> +);
> +
>  sub trim { my $s = shift; $s =~ s/^\s+|\s+$//g; return $s };
>  
>  sub print_help {
> @@ -29,7 +35,7 @@ sub print_help {
>  my $print_full = 1;
>  my $print_names = 0;
>  my $print_emails = 0;
> -my $date         = DateTime->now()->iso8601;
> +my $date_str     = DateTime->now()->iso8601;
>  my $help = 0;
>  
>  GetOptions(
> @@ -37,7 +43,7 @@ GetOptions(
>      "names" => \$print_names,
>      "emails" => \$print_emails,
>      "help" => \$help,
> -    "date=s" => \$date,
> +    "date=s" => \$date_str,
>      "h" => \$help,
>  );
>  
> @@ -76,7 +82,8 @@ sub get_date_range {
>      return ($date_since, $date_until);
>  }
>  
> -my ($since, $until) = get_date_range(DateTime::Format::ISO8601->parse_datetime($date));
> +my $date = DateTime::Format::ISO8601->parse_datetime($date_str);
> +my ($since, $until) = get_date_range($date);
>  
>  my @shortlog = split /\n/, decode('UTF-8',
>      `git log --pretty=format:"%aN <%aE>" --since="$since" --until="$until" | sort | uniq -c | sort -r`,
> @@ -108,6 +115,13 @@ foreach my $line (@shortlog) {
>      $assembly{$name} = $email;
>  }
>  
> +foreach my $entry (@extra_members) {
> +    my $elected = $entry->[2];
> +    if ($date->is_between($elected, $elected->clone()->set_year($elected->year + 2))) {

tools/general_assembly.pl
Can't locate object method "is_between" via package "DateTime" at tools/general_assembly.pl line 75.

is_between seems to have been added in 1.52   2020-02-29
So this is unavailable in several distros, it works with cpan though
but this shoould be docuemnted if it cannot be avoided

thx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Concerning the gods, I have no means of knowing whether they exist or not
or of what sort they may be, because of the obscurity of the subject, and
the brevity of human life -- Protagoras

[-- Attachment #1.2: signature.asc --]
[-- Type: application/pgp-signature, Size: 195 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".