Git development
 help / color / mirror / Atom feed
* [PATCH] gitattributes: fix eol attribute for Perl scripts
@ 2026-06-15  4:25 Koutian Wu via GitGitGadget
  2026-06-15  7:22 ` Patrick Steinhardt
  2026-06-15  7:53 ` [PATCH v2] " Koutian Wu via GitGitGadget
  0 siblings, 2 replies; 4+ messages in thread
From: Koutian Wu via GitGitGadget @ 2026-06-15  4:25 UTC (permalink / raw)
  To: git; +Cc: Koutian Wu, ktwu01

From: ktwu01 <ktwu01@gmail.com>

The *.pl pattern currently sets eof=lf, which is not a built-in
attribute used for line-ending normalization.

Use eol=lf instead, matching the neighboring *.perl and *.pm rules, so
Perl scripts are checked out with LF line endings.

Signed-off-by: ktwu01 <ktwu01@gmail.com>
---
    gitattributes: fix eol attribute for Perl scripts

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-2151%2Fktwu01%2Fkw%2Ffix-pl-eol-attribute-v1
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-2151/ktwu01/kw/fix-pl-eol-attribute-v1
Pull-Request: https://github.com/gitgitgadget/git/pull/2151

 .gitattributes | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitattributes b/.gitattributes
index 556322be01..26490ad60a 100644
--- a/.gitattributes
+++ b/.gitattributes
@@ -2,7 +2,7 @@
 *.[ch] whitespace=indent,trail,space,incomplete diff=cpp
 *.sh whitespace=indent,trail,space,incomplete text eol=lf
 *.perl text eol=lf diff=perl
-*.pl text eof=lf diff=perl
+*.pl text eol=lf diff=perl
 *.pm text eol=lf diff=perl
 *.py text eol=lf diff=python
 *.bat text eol=crlf

base-commit: ea97ad8d017de0c9037451a78008a0fd60abea0c
-- 
gitgitgadget

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] gitattributes: fix eol attribute for Perl scripts
  2026-06-15  4:25 [PATCH] gitattributes: fix eol attribute for Perl scripts Koutian Wu via GitGitGadget
@ 2026-06-15  7:22 ` Patrick Steinhardt
  2026-06-15  7:53 ` [PATCH v2] " Koutian Wu via GitGitGadget
  1 sibling, 0 replies; 4+ messages in thread
From: Patrick Steinhardt @ 2026-06-15  7:22 UTC (permalink / raw)
  To: Koutian Wu via GitGitGadget; +Cc: git, Koutian Wu

On Mon, Jun 15, 2026 at 04:25:25AM +0000, Koutian Wu via GitGitGadget wrote:
> From: ktwu01 <ktwu01@gmail.com>
> 
> The *.pl pattern currently sets eof=lf, which is not a built-in
> attribute used for line-ending normalization.
> 
> Use eol=lf instead, matching the neighboring *.perl and *.pm rules, so
> Perl scripts are checked out with LF line endings.
> 
> Signed-off-by: ktwu01 <ktwu01@gmail.com>

The Signed-off-by and commit author should use your real name, if
possible. See [1].

> diff --git a/.gitattributes b/.gitattributes
> index 556322be01..26490ad60a 100644
> --- a/.gitattributes
> +++ b/.gitattributes
> @@ -2,7 +2,7 @@
>  *.[ch] whitespace=indent,trail,space,incomplete diff=cpp
>  *.sh whitespace=indent,trail,space,incomplete text eol=lf
>  *.perl text eol=lf diff=perl
> -*.pl text eof=lf diff=perl
> +*.pl text eol=lf diff=perl
>  *.pm text eol=lf diff=perl
>  *.py text eol=lf diff=python
>  *.bat text eol=crlf

Yeah, this looks obviously correct to me. Thanks for the fix!

Patrick

[1]: https://git-scm.com/docs/SubmittingPatches#real-name

^ permalink raw reply	[flat|nested] 4+ messages in thread

* [PATCH v2] gitattributes: fix eol attribute for Perl scripts
  2026-06-15  4:25 [PATCH] gitattributes: fix eol attribute for Perl scripts Koutian Wu via GitGitGadget
  2026-06-15  7:22 ` Patrick Steinhardt
@ 2026-06-15  7:53 ` Koutian Wu via GitGitGadget
  2026-06-15  8:37   ` Patrick Steinhardt
  1 sibling, 1 reply; 4+ messages in thread
From: Koutian Wu via GitGitGadget @ 2026-06-15  7:53 UTC (permalink / raw)
  To: git; +Cc: Koutian Wu, Koutian Wu

From: Koutian Wu <ktwu01@gmail.com>

The *.pl pattern currently sets eof=lf, which is not a built-in
attribute used for line-ending normalization.

Use eol=lf instead, matching the neighboring *.perl and *.pm rules, so
Perl scripts are checked out with LF line endings.

Signed-off-by: Koutian Wu <ktwu01@gmail.com>
---
    gitattributes: fix eol attribute for Perl scripts

Published-As: https://github.com/gitgitgadget/git/releases/tag/pr-2151%2Fktwu01%2Fkw%2Ffix-pl-eol-attribute-v2
Fetch-It-Via: git fetch https://github.com/gitgitgadget/git pr-2151/ktwu01/kw/fix-pl-eol-attribute-v2
Pull-Request: https://github.com/gitgitgadget/git/pull/2151

Range-diff vs v1:

 1:  92ba4d499d ! 1:  f4b4ca30c7 gitattributes: fix eol attribute for Perl scripts
     @@
       ## Metadata ##
     -Author: ktwu01 <ktwu01@gmail.com>
     +Author: Koutian Wu <ktwu01@gmail.com>
      
       ## Commit message ##
          gitattributes: fix eol attribute for Perl scripts
     @@ Commit message
          Use eol=lf instead, matching the neighboring *.perl and *.pm rules, so
          Perl scripts are checked out with LF line endings.
      
     -    Signed-off-by: ktwu01 <ktwu01@gmail.com>
     +    Signed-off-by: Koutian Wu <ktwu01@gmail.com>
      
       ## .gitattributes ##
      @@


 .gitattributes | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/.gitattributes b/.gitattributes
index 556322be01..26490ad60a 100644
--- a/.gitattributes
+++ b/.gitattributes
@@ -2,7 +2,7 @@
 *.[ch] whitespace=indent,trail,space,incomplete diff=cpp
 *.sh whitespace=indent,trail,space,incomplete text eol=lf
 *.perl text eol=lf diff=perl
-*.pl text eof=lf diff=perl
+*.pl text eol=lf diff=perl
 *.pm text eol=lf diff=perl
 *.py text eol=lf diff=python
 *.bat text eol=crlf

base-commit: ea97ad8d017de0c9037451a78008a0fd60abea0c
-- 
gitgitgadget

^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH v2] gitattributes: fix eol attribute for Perl scripts
  2026-06-15  7:53 ` [PATCH v2] " Koutian Wu via GitGitGadget
@ 2026-06-15  8:37   ` Patrick Steinhardt
  0 siblings, 0 replies; 4+ messages in thread
From: Patrick Steinhardt @ 2026-06-15  8:37 UTC (permalink / raw)
  To: Koutian Wu via GitGitGadget; +Cc: git, Koutian Wu

On Mon, Jun 15, 2026 at 07:53:58AM +0000, Koutian Wu via GitGitGadget wrote:
> Range-diff vs v1:
> 
>  1:  92ba4d499d ! 1:  f4b4ca30c7 gitattributes: fix eol attribute for Perl scripts
>      @@
>        ## Metadata ##
>      -Author: ktwu01 <ktwu01@gmail.com>
>      +Author: Koutian Wu <ktwu01@gmail.com>
>       
>        ## Commit message ##
>           gitattributes: fix eol attribute for Perl scripts
>      @@ Commit message
>           Use eol=lf instead, matching the neighboring *.perl and *.pm rules, so
>           Perl scripts are checked out with LF line endings.
>       
>      -    Signed-off-by: ktwu01 <ktwu01@gmail.com>
>      +    Signed-off-by: Koutian Wu <ktwu01@gmail.com>
>       
>        ## .gitattributes ##
>       @@

Thanks, this version looks good to me!

Patrick

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2026-06-15  8:37 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2026-06-15  4:25 [PATCH] gitattributes: fix eol attribute for Perl scripts Koutian Wu via GitGitGadget
2026-06-15  7:22 ` Patrick Steinhardt
2026-06-15  7:53 ` [PATCH v2] " Koutian Wu via GitGitGadget
2026-06-15  8:37   ` Patrick Steinhardt

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox