| From: | Daniel Gustafsson <daniel(at)yesql(dot)se> |
|---|---|
| To: | Erik Wienhold <ewie(at)ewie(dot)name> |
| Cc: | PostgreSQL Hackers <pgsql-hackers(at)postgresql(dot)org> |
| Subject: | Re: pgindent fails with perl 5.40 |
| Date: | 2024-10-08 09:26:13 |
| Message-ID: | 2EFDAF8C-0ED7-47A3-BD50-51DE29D279DF@yesql.se |
| Views: | Whole Thread | Raw Message | Download mbox | Resend email |
| Thread: | |
| Lists: | pgsql-hackers |
> On 8 Oct 2024, at 03:50, Erik Wienhold <ewie(at)ewie(dot)name> wrote:
>> Calling the import method of an unknown package produces a warning
>> [...]
>> It will also detect cases where a user passes an argument when using a
>> package that does not provide its own import
>> [...]
>
> Because we use fatal warnings, pgindent fails.
I'm far from fluent in Perl, but my reading of this is that File::Spec doesn't
provide an import() function so explicitly listing devnull does nothing here.
Your diff should thus be a safe fix. The devnull function existed already in
3.00 which was released just a hair over 2 decades ago so I think we can safely
assume any File::Spec we use to have it.
--
Daniel Gustafsson
| From | Date | Subject | |
|---|---|---|---|
| Next Message | Nisha Moond | 2024-10-08 09:39:24 | Re: Conflict Detection and Resolution |
| Previous Message | Andrei Lepikhov | 2024-10-08 09:22:36 | Re: Expand applicability of aggregate's sortop optimization |