Hi all,

This email starts a discussion about an important proposal for our
project's future: changing the C code indentation style from tabs to
spaces.

We know that tabs have been the rsyslog standard for a very long time,
and this is not a change to be taken lightly. The goal is not
aesthetic, but pragmatic, aimed at improving our development process.

== The Problem ==

We face growing friction from two sources:

New Contributors: Many new developers submit patches with spaces,
which is the default in most modern editors. This requires extra work
from maintainers to fix and can be a frustrating first experience for
contributors.

Modern Tooling: AI-powered tools like GitHub Copilot and other code
generators are heavily optimized for space-indented code. They often
struggle to produce correct, tab-indented code, which limits our
ability to use these tools effectively.

== The Proposed Solution ==

The proposal is to reformat the entire codebase to use 4 spaces for
indentation in a single, automated commit.

A major concern is preserving our git blame history. We will solve
this using a modern Git feature: the .git-blame-ignore-revs file.

By adding the reformatting commit's hash to this file, we tell Git to
"look past" it when running blame. GitHub's web UI supports this
automatically, so our history will remain fully intact and browsable.
For local use, developers would only need to run a simple one-time
config command.

== Call for Discussion ==

We believe this change will lower the barrier for new contributors and
allow us all to better leverage modern tools. However, a change of
this nature requires community consensus.

Please share your thoughts and concerns on this list.

For a more detailed breakdown of the plan and to follow the full
discussion, please see the GitHub issue here:

https://github.com/rsyslog/rsyslog/issues/5747

Thanks,
Rainer
_______________________________________________
rsyslog mailing list
https://lists.adiscon.net/mailman/listinfo/rsyslog
http://www.rsyslog.com/professional-services/
What's up with rsyslog? Follow https://twitter.com/rgerhards
NOTE WELL: This is a PUBLIC mailing list, posts are ARCHIVED by a myriad of 
sites beyond our control. PLEASE UNSUBSCRIBE and DO NOT POST if you DON'T LIKE 
THAT.

Reply via email to