SSL TAP tests and chmod

Started by Michael Paquieralmost 11 years ago3 messageshackers
Jump to latest
#1Michael Paquier
michael@paquier.xyz

Hi all,

The TAP tests in src/test/ssl are using system() in combination with
chmod, but perl has a command chmod integrated into it, and it would
work better on Windows as well.
The attached patch is aimed at fixing that.

Regards,
--
Michael

Attachments:

20150619_ssl_tap_chmod_fix.patchapplication/x-patch; name=20150619_ssl_tap_chmod_fix.patchDownload+16-2
#2Michael Paquier
michael@paquier.xyz
In reply to: Michael Paquier (#1)
Re: SSL TAP tests and chmod

On Fri, Jun 19, 2015 at 2:49 PM, Michael Paquier wrote:

The TAP tests in src/test/ssl are using system() in combination with
chmod, but perl has a command chmod integrated into it, and it would
work better on Windows as well.
The attached patch is aimed at fixing that.

For the sake of the archives, this has been committed by Robert (thanks!):
commit: ca3f43aa48a83013ea50aeee7cd193a5859c4587
author: Robert Haas <rhaas@postgresql.org>
date: Fri, 19 Jun 2015 10:46:30 -0400
Change TAP test framework to not rely on having a chmod executable.

This might not work at all on Windows, and is not ever efficient.

Regards,
--
Michael

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers

#3Robert Haas
robertmhaas@gmail.com
In reply to: Michael Paquier (#2)
Re: SSL TAP tests and chmod

On Sun, Jun 21, 2015 at 9:50 PM, Michael Paquier
<michael.paquier@gmail.com> wrote:

On Fri, Jun 19, 2015 at 2:49 PM, Michael Paquier wrote:

The TAP tests in src/test/ssl are using system() in combination with
chmod, but perl has a command chmod integrated into it, and it would
work better on Windows as well.
The attached patch is aimed at fixing that.

For the sake of the archives, this has been committed by Robert (thanks!):
commit: ca3f43aa48a83013ea50aeee7cd193a5859c4587
author: Robert Haas <rhaas@postgresql.org>
date: Fri, 19 Jun 2015 10:46:30 -0400
Change TAP test framework to not rely on having a chmod executable.

This might not work at all on Windows, and is not ever efficient.

Whoops, sorry, I thought I had replied to this thread, but I guess I forgot.

--
Robert Haas
EnterpriseDB: http://www.enterprisedb.com
The Enterprise PostgreSQL Company

--
Sent via pgsql-hackers mailing list (pgsql-hackers@postgresql.org)
To make changes to your subscription:
http://www.postgresql.org/mailpref/pgsql-hackers