[FFmpeg-devel] Forgejo entry threshold

[FFmpeg-devel] Forgejo entry threshold

[Nicolas George]

Dummy_test_user (HE12025-08-04):
> PR #20112 opened by Dummy_test_user
> URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20112
> Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20112.patch
> 
> I did that pull request to see what is preventing me from submitting crappy code.
> Apparently nothing.

Hi.

That was me. Thank you for ensuring it went to the mailing-list.

But…

I could register an account and create a pull request without ever
reading the coding-style rules in doc/developer.texi, and the pull
requests has red patcheck:

mismatching doxy params
+ * @param erbut  useless
+ * @param ernut  does nothing

That is not good.

If nothing better comes to mind to exclude crappy code, could we at
least condition creating an account to answering a 10 questions quiz
about our coding conventions?

Regards,

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Forgejo entry threshold

[Hendrik Leppkes]

On Mon, Aug 4, 2025 at 4:45 PM Nicolas George <george@nsup.org> wrote:
>
> Dummy_test_user (HE12025-08-04):
> > PR #20112 opened by Dummy_test_user
> > URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20112
> > Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20112.patch
> >
> > I did that pull request to see what is preventing me from submitting crappy code.
> > Apparently nothing.
>
> Hi.
>
> That was me. Thank you for ensuring it went to the mailing-list.
>
> But…
>
> I could register an account and create a pull request without ever
> reading the coding-style rules in doc/developer.texi,
>

Whats the barrier of entry to sending a patch to the ML right now?
Does it quiz you on the guidelines?

- Hendrik
_______________________________________________
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".

Re: [FFmpeg-devel] Forgejo entry threshold

[Nicolas George]

Hendrik Leppkes (HE12025-08-04):
> Whats the barrier of entry to sending a patch to the ML right now?

Being able to, which is strongly correlated to knowing good practices.

> Does it quiz you on the guidelines?

Do you have better to propose?

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Forgejo entry threshold

[Leon Grutters]

On 8/4/25 4:45 PM, Nicolas George wrote:
> Dummy_test_user (HE12025-08-04):
>> PR #20112 opened by Dummy_test_user
>> URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20112
>> Patch URL: https://code.ffmpeg.org/FFmpeg/FFmpeg/pulls/20112.patch
>>
>> I did that pull request to see what is preventing me from submitting crappy code.
>> Apparently nothing.
> Hi.
>
> That was me. Thank you for ensuring it went to the mailing-list.
>
> But…
>
> I could register an account and create a pull request without ever
> reading the coding-style rules in doc/developer.texi, and the pull
> requests has red patcheck:
>
> mismatching doxy params
> + * @param erbut  useless
> + * @param ernut  does nothing
>
> That is not good.
>
> If nothing better comes to mind to exclude crappy code, could we at
> least condition creating an account to answering a 10 questions quiz
> about our coding conventions?
>
> Regards,
>
Is it that much harder to submit a patch on the ML? All you do is sign 
up for the ML, run one git command and then attach it to an email and 
click send.

_______________________________________________
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".

Re: [FFmpeg-devel] Forgejo entry threshold

[Nicolas George]

Leon Grutters (HE12025-08-04):
> Is it that much harder to submit a patch on the ML? All you do is sign up
> for the ML, run one git command and then attach it to an email and click
> send.

No, it is not harder at all, but it has the reputation to be, which has
the same result.

Nowadays, any idiot wannabe coder already has a github account and knows
how to fumble on a similar webapp without having ever heard of
etiquette, coding style or good practices. At least, when learning
involves reading a doc instead of clicking randomly on buttons, they
would have an occasion to.

Regards,

-- 
  Nicolas George
_______________________________________________
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".

Re: [FFmpeg-devel] Forgejo entry threshold

[Kacper Michajlow]

On Tue, 5 Aug 2025 at 15:10, Nicolas George <george@nsup.org> wrote:
>
> Leon Grutters (HE12025-08-04):
> > Is it that much harder to submit a patch on the ML? All you do is sign up
> > for the ML, run one git command and then attach it to an email and click
> > send.
>
> No, it is not harder at all, but it has the reputation to be, which has
> the same result.
>
> Nowadays, any idiot wannabe coder already has a github account and knows
> how to fumble on a similar webapp without having ever heard of
> etiquette, coding style or good practices. At least, when learning
> involves reading a doc instead of clicking randomly on buttons, they
> would have an occasion to.

Those "idiot wannabe coders," as you call them, are simply not
interested in low-level C/ASM media processing libraries. They use
Playskool Plastic Tool Kit to develop web applications in javascript.

Seriously, I think elitism and gatekeeping are not the right approach.
Judging someone's contribution quality based on whether they use gmail
or other "non-approved" tools is unfair. It saddens me that this kind
of attitude comes from a senior member of the community with so much
experience in the project.

The whole point is to attract new contributors and make the workflow
easier for everyone to improve productivity. If you care about
quality, go review the patches.

- Kacper
_______________________________________________
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".

Re: [FFmpeg-devel] Forgejo entry threshold

[Nicolas George]

Nicolas George (HE12025-08-04):
> That was me.

Whoever disabled the account: I went to just delete it, and now I
cannot, so the joke is on you, you have to do it yourself.

> Thank you for ensuring it went to the mailing-list.

It seems to work randomly.

Regards,

-- 
  Nicolas George
_______________________________________________
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".