Movatterモバイル変換

[0]ホーム
URL:

Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

 Sign up
Appearance settings

Commit99cd749

Browse files
Revert "Skip .DS_Store files in server side utils"
This reverts commitaeee173.Per failure reports from the buildfarm.
1 parentaeee173 commit99cd749

File tree

9 files changed

+4
-46
lines changed

9 files changed

+4
-46
lines changed

‎doc/src/sgml/protocol.sgml‎

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2842,7 +2842,7 @@ The commands accepted in replication mode are:
28422842
<para>
28432843
Files other than regular files and directories, such as symbolic
28442844
links (other than for the directories listed above) and special
2845-
deviceand operating systemfiles, are skipped. (Symbolic links
2845+
device files, are skipped. (Symbolic links
28462846
in <filename>pg_tblspc</filename> are maintained.)
28472847
</para>
28482848
</listitem>

‎doc/src/sgml/ref/pg_basebackup.sgml‎

Lines changed: 1 addition & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -832,8 +832,7 @@ PostgreSQL documentation
832832
The backup will include all files in the data directory and tablespaces,
833833
including the configuration files and any additional files placed in the
834834
directory by third parties, except certain temporary files managed by
835-
PostgreSQL and operating system files. But only regular files and
836-
directories are copied, except that
835+
PostgreSQL. But only regular files and directories are copied, except that
837836
symbolic links used for tablespaces are preserved. Symbolic links pointing
838837
to certain directories known to PostgreSQL are copied as empty directories.
839838
Other symbolic links and special device files are skipped.

‎doc/src/sgml/ref/pg_rewind.sgml‎

Lines changed: 2 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -374,9 +374,8 @@ GRANT EXECUTE ON function pg_catalog.pg_read_binary_file(text, bigint, bigint, b
374374
<filename>backup_label</filename>,
375375
<filename>tablespace_map</filename>,
376376
<filename>pg_internal.init</filename>,
377-
<filename>postmaster.opts</filename>,
378-
<filename>postmaster.pid</filename> and
379-
<filename>.DS_Store</filename> as well as any file or directory
377+
<filename>postmaster.opts</filename>, and
378+
<filename>postmaster.pid</filename>, as well as any file or directory
380379
beginning with <filename>pgsql_tmp</filename>, are omitted.
381380
</para>
382381
</step>

‎src/backend/replication/basebackup.c‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -1272,10 +1272,6 @@ sendDir(const char *path, int basepathlen, bool sizeonly, List *tablespaces,
12721272
strlen(PG_TEMP_FILE_PREFIX))==0)
12731273
continue;
12741274

1275-
/* Skip macOS system files */
1276-
if (strcmp(de->d_name,".DS_Store")==0)
1277-
continue;
1278-
12791275
/*
12801276
* Check if the postmaster has signaled us to exit, and abort with an
12811277
* error in that case. The error handler further up will call

‎src/bin/pg_basebackup/t/010_pg_basebackup.pl‎

Lines changed: 0 additions & 16 deletions
Original file line numberDiff line numberDiff line change
@@ -76,16 +76,6 @@
7676
close$file;
7777
}
7878

79-
# Test that macOS system files are skipped. Only test on non-macOS systems
80-
# however since creating incorrect .DS_Store files on a macOS system may have
81-
# unintended side effects.
82-
if ($Config{osname}ne'darwin')
83-
{
84-
openmy$file,'>>',"$pgdata/.DS_Store";
85-
print$file"DONOTCOPY";
86-
close$file;
87-
}
88-
8979
# Connect to a database to create global/pg_internal.init. If this is removed
9080
# the test to ensure global/pg_internal.init is not copied will return a false
9181
# positive.
@@ -154,12 +144,6 @@
154144
ok(!-f"$tempdir/backup/$filename","$filename not copied");
155145
}
156146

157-
# We only test .DS_Store files being skipped on non-macOS systems
158-
if ($Config{osname}ne'darwin')
159-
{
160-
ok(!-f"$tempdir/backup/.DS_Store",".DS_Store not copied");
161-
}
162-
163147
# Unlogged relation forks other than init should not be copied
164148
ok(-f"$tempdir/backup/${baseUnloggedPath}_init",
165149
'unlogged init fork in backup');

‎src/bin/pg_checksums/pg_checksums.c‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -338,10 +338,6 @@ scan_directory(const char *basedir, const char *subdir, bool sizeonly)
338338
strlen(PG_TEMP_FILES_DIR))==0)
339339
continue;
340340

341-
/* Skip macOS system files */
342-
if (strcmp(de->d_name,".DS_Store")==0)
343-
continue;
344-
345341
snprintf(fn,sizeof(fn),"%s/%s",path,de->d_name);
346342
if (lstat(fn,&st)<0)
347343
{

‎src/bin/pg_checksums/t/002_actions.pl‎

Lines changed: 0 additions & 7 deletions
Original file line numberDiff line numberDiff line change
@@ -6,7 +6,6 @@
66

77
use strict;
88
use warnings;
9-
use Config;
109
use PostgresNode;
1110
use TestLib;
1211

@@ -115,12 +114,6 @@ sub check_relation_corruption
115114
append_to_file"$pgdata/global/pg_internal.init","foo";
116115
append_to_file"$pgdata/global/pg_internal.init.123","foo";
117116

118-
# These are non-postgres macOS files, which should be ignored by the scan.
119-
# Only perform this test on non-macOS systems though as creating incorrect
120-
# system files may have side effects on macOS.
121-
append_to_file"$pgdata/global/.DS_Store","foo"
122-
unless ($Config{osname}eq'darwin');
123-
124117
# Enable checksums.
125118
command_ok(['pg_checksums','--enable','--no-sync','-D',$pgdata ],
126119
"checksums successfully enabled in cluster");

‎src/bin/pg_rewind/filemap.c‎

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -648,10 +648,6 @@ decide_file_action(file_entry_t *entry)
648648
if (strcmp(path,"global/pg_control")==0)
649649
returnFILE_ACTION_NONE;
650650

651-
/* Skip macOS system files */
652-
if (strstr(path,".DS_Store")!=NULL)
653-
returnFILE_ACTION_NONE;
654-
655651
/*
656652
* Remove all files matching the exclusion filters in the target.
657653
*/

‎src/bin/pg_rewind/t/003_extrafiles.pl‎

Lines changed: 0 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -5,7 +5,6 @@
55

66
use strict;
77
use warnings;
8-
use Config;
98
use TestLib;
109
use Test::Moretests=> 5;
1110

@@ -54,10 +53,6 @@ sub run_test
5453
append_to_file
5554
"$test_standby_datadir/tst_standby_dir/standby_subdir/standby_file4",
5655
"in standby4";
57-
# Skip testing .DS_Store files on macOS to avoid risk of side effects
58-
append_to_file
59-
"$test_standby_datadir/tst_standby_dir/.DS_Store",
60-
"macOS system file"unless ($Config{osname}eq'darwin');
6156

6257
mkdir"$test_primary_datadir/tst_primary_dir";
6358
append_to_file"$test_primary_datadir/tst_primary_dir/primary_file1",

0 commit comments

Comments
 (0)
[8]ページ先頭
©2009-2025 Movatter.jp