npm-pkgbuild 7.3.8 → 7.3.12

package/package.json

@@ -1,6 +1,6 @@
 {
   "name": "npm-pkgbuild",
-  "version": "7.3.8",
+  "version": "7.3.12",
   "publishConfig": {
     "access": "public"
   },
@@ -40,8 +40,8 @@
   "dependencies": {
     "aggregate-async-iterator": "^1.1.7",
     "commander": "^8.3.0",
-    "content-entry": "^2.9.6",
-    "content-entry-filesystem": "^3.1.9",
+    "content-entry": "^2.10.0",
+    "content-entry-filesystem": "^3.1.11",
     "execa": "^6.0.0",
     "expression-expander": "^7.0.9",
     "globby": "^12.0.2",

package/src/output/deb.mjs

@@ -1,21 +1,15 @@
 import { join, dirname } from "path";
 import { tmpdir } from "os";
-import { createWriteStream } from "fs";
-import { mkdtemp, mkdir, chmod } from "fs/promises";
-import { pipeline } from "stream/promises";
+import { mkdtemp } from "fs/promises";
 import { execa } from "execa";
 import { EmptyContentEntry, ReadableStreamContentEntry } from "content-entry";
 import { keyValueTransformer } from "key-value-transformer";
 import { Packager } from "./packager.mjs";
+import { copyEntries, transform } from "../util.mjs";
 
-const executableAttributes = { chmod: "0775" };
-
-const permissions = {
-  "DEBIAN/preinst": executableAttributes,
-  "DEBIAN/postinst": executableAttributes,
-  "DEBIAN/prerm": executableAttributes,
-  "DEBIAN/postrm": executableAttributes
-};
+const attributes = [
+  { pattern: /DEBIAN\/.*(inst|rm)/, mode: 0o775 }
+];
 
 export class DEB extends Packager {
   static get name() {
@@ -62,47 +56,14 @@ export class DEB extends Packager {
             entry.name,
             keyValueTransformer(await entry.readStream, controlProperties)
           ),
-        createEntryWhenMissing: new EmptyContentEntry(debianControlName)
+        createEntryWhenMissing: () => new EmptyContentEntry(debianControlName)
       }
     ];
 
-    let debianControlEntry;
-
-    for await (const entry of this.source) {
-      const destName = join(staging, entry.name);
-
-      await mkdir(dirname(destName), { recursive: true });
-
-      if (entry.name === debianControlName) {
-        debianControlEntry = entry;
-      } else {
-        console.log("ENTRY", entry.name, entry.basename);
-        await pipeline(await entry.readStream, createWriteStream(destName));
-
-        await Promise.all(
-          Object.entries(permissions).map(async ([pattern, option]) => {
-            if (destName.endsWith(pattern)) {
-              return chmod(destName, option.chmod);
-            }
-          })
-        );
-      }
-    }
-
-    if (!debianControlEntry) {
-      debianControlEntry = new EmptyContentEntry(debianControlName);
-    }
-
-    let destName = join(staging, debianControlEntry.name);
-
-    await mkdir(dirname(destName), { recursive: true });
-
-    await pipeline(
-      keyValueTransformer(
-        await debianControlEntry.readStream,
-        controlProperties
-      ),
-      createWriteStream(destName)
+    await copyEntries(
+      transform(this.source, transformers),
+      staging,
+      attributes
     );
 
     await execa("dpkg", ["-b", staging, options.destination]);
@@ -113,7 +74,7 @@ export class DEB extends Packager {
 
 /**
  * @see https://www.debian.org/doc/debian-policy/ch-controlfields.html
- * @ https://linux.die.net/man/5/deb-control
+ * @see https://linux.die.net/man/5/deb-control
  */
 
 const fields = {

package/src/output/pkg.mjs

@@ -22,12 +22,20 @@ export class PKG extends Packager {
   }
 
   async execute(options) {
-    const tmp = await mkdtemp(join(tmpdir(), "pkg-"));
+    const properties = this.properties;
+    const mandatoryFields = this.mandatoryFields;
+    const tmp = await mkdtemp(join(tmpdir(), "deb-"));
+    const staging = join(tmp, `${properties.name}-${properties.version}`);
+
 
     const pkgbuildFileName = join(tmp, "PKGBUILD");
 
     this.writePkbuild(pkgbuildFileName);
 
+    const transformers = [];
+
+    await copyEntries(transform(this.source, transformers), staging);
+
     await execa("makepkg", [], { cwd: tmp });
   }
 

package/src/util.mjs

@@ -66,25 +66,57 @@ export function extractFromPackage(pkg) {
   return { properties, sources };
 }
 
+/**
+ * Apply transformers
+ * @param {AsyncIterator<ContentEntry>} source
+ * @param {Transformer[]} transformers
+ */
+export async function* transform(source, transformers) {
+  const usedTransformers = new Set();
+
+  for await (let entry of source) {
+    for (const t of transformers) {
+      if (t.match(entry)) {
+        entry = await t.transform(entry);
+        usedTransformers.add(t);
+      }
+    }
+
+    yield entry;
+  }
+
+  for (const t of transformers) {
+    if (!usedTransformers.has(t) && t.createEntryWhenMissing !== undefined) {
+      yield t.transform(await t.createEntryWhenMissing());
+    }
+  }
+}
+
 /**
  * Copy content from source into destinationDirectory
  * @param {AsyncIterator<ContentEntry>} source
  * @param {string} destinationDirectory
- * @param {Transformer[]} transformers
  */
-export async function copyEntries(source, destinationDirectory, transformers) {
+export async function copyEntries(
+  source,
+  destinationDirectory,
+  attributes = []
+) {
   for await (let entry of source) {
     const destName = join(destinationDirectory, entry.name);
-
     await mkdir(dirname(destName), { recursive: true });
 
-    for (const t of transformers) {
-      if (t.match(entry)) {
-        entry = await t.transform(entry);
-        break;
+    const options = { mode: entry.mode };
+
+    for (const a of attributes) {
+      if (entry.name.match(a.pattern)) {
+        options.mode = a.mode;
       }
     }
 
-    await pipeline(await entry.readStream, createWriteStream(destName));
+    await pipeline(
+      await entry.readStream,
+      createWriteStream(destName, options)
+    );
   }
 }