Skip to content
This repository was archived by the owner on Apr 25, 2024. It is now read-only.

Commit 3bc4c5f

Browse files
committed
Merge pull request github#34 from gautaz/master
fixes github#31
2 parents f173c0f + 93d9509 commit 3bc4c5f

File tree

5 files changed

+17
-2
lines changed

5 files changed

+17
-2
lines changed

index.js

Lines changed: 4 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -100,22 +100,24 @@ exports.pack = function (cwd, opts) {
100100
gid: stat.gid
101101
}
102102

103-
header = map(header) || header
104-
105103
if (stat.isDirectory()) {
106104
header.size = 0
107105
header.type = 'directory'
106+
header = map(header) || header
108107
return pack.entry(header, onnextentry)
109108
}
110109

111110
if (stat.isSymbolicLink()) {
112111
header.size = 0
113112
header.type = 'symlink'
113+
header = map(header) || header
114114
return onlink(filename, header)
115115
}
116116

117117
// TODO: add fifo etc...
118118

119+
header = map(header) || header
120+
119121
if (!stat.isFile()) {
120122
if (strict) return pack.destroy(new Error('unsupported type for ' + filename))
121123
return onnextentry()

test/fixtures/e/directory/.ignore

Whitespace-only changes.

test/fixtures/e/file

Whitespace-only changes.

test/fixtures/e/symlink

Lines changed: 1 addition & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1 @@
1+
symlink

test/index.js

Lines changed: 12 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -213,3 +213,15 @@ test('specific entries', function (t) {
213213
t.same(subDir, ['file5'])
214214
})
215215
})
216+
217+
test('check type while mapping header on packing', function (t) {
218+
t.plan(3)
219+
220+
var e = path.join(__dirname, 'fixtures', 'e')
221+
222+
var checkHeaderType = function (header) {
223+
if (header.name.indexOf('.') === -1) t.same(header.type, header.name)
224+
}
225+
226+
tar.pack(e, { map: checkHeaderType })
227+
})

0 commit comments

Comments
 (0)