Remove specific _\n code in TAP for Windows

Поиск
Список
Период
Сортировка
От Michael Paquier
Тема Remove specific _\n code in TAP for Windows
Дата
Msg-id aPsh39bxwYKvUlAf@paquier.xyz
обсуждение исходный текст
Ответы Re: Remove specific _\n code in TAP for Windows
Список pgsql-hackers
Hi all,

$subject has been mentioned by Jacob (in CC.) on Discord, and it does
not seem like somebody took the time to propose a patch.

We use in some places of the TAP tests the following pattern:
my $newline = $windows_os ? "\r\n" : "\n";

Two files have this idea:
- 003_ldap_connection_param_lookup.pl
- 006_service.pl
Note: I have committed one of these.

However, in light of 1c6d4629394d, we should not require that, and the
CI looks happy with the attached.

That would be up to the buildfarm to act as final judge, but any
objections in attempting to get rid of these like in the attached?
I would try first HEAD to be sure, then follow with an optional
backpatch.

Thoughts?
--
Michael

Вложения

В списке pgsql-hackers по дате отправления: