Skip to content

Commit 0d9fa3b

Browse files
committed
Ensure that all required packaged files have extensions so that vsixsign
is valid. 1. Added .sh extension to: ant, antRun, mvnDebug, mvnjyp. 2. Added .txt extension to: NOTICE, LICENSE, project-marker-jdk. 3. Ignore nb clusters .lastModified files and node_modules .npmignore from vsix packaging. Signed-off-by: Siddharth Srinivasan <[email protected]>
1 parent 40d9fe2 commit 0d9fa3b

File tree

3 files changed

+47
-5
lines changed

3 files changed

+47
-5
lines changed

build.xml

Lines changed: 27 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -63,6 +63,7 @@
6363
patches/8036-draft.diff
6464
patches/8038-draft.diff
6565
patches/mvn-sh.diff
66+
patches/project-marker-jdk.diff
6667
patches/generate-dependencies.diff
6768
patches/rename-debugger.diff
6869
patches/remove-db.diff
@@ -109,7 +110,11 @@
109110
<mkdir dir="${lsp.build.dir}/etc" />
110111
<copy todir="${lsp.build.dir}/etc" file="script/etc/nbcode.conf" overwrite="true" />
111112
<copy todir="${lsp.build.dir}/etc" file="script/etc/nbcode.clusters" overwrite="true" />
112-
<move file="${lsp.build.dir}/java/maven/bin/mvn" tofile="${lsp.build.dir}/java/maven/bin/mvn.sh" />
113+
<move file="${lsp.build.dir}/extide/ant/bin/ant" tofile="${lsp.build.dir}/extide/ant/bin/ant.sh"/>
114+
<move file="${lsp.build.dir}/extide/ant/bin/antRun" tofile="${lsp.build.dir}/extide/ant/bin/antRun.sh"/>
115+
<move file="${lsp.build.dir}/java/maven/bin/mvn" tofile="${lsp.build.dir}/java/maven/bin/mvn.sh"/>
116+
<move file="${lsp.build.dir}/java/maven/bin/mvnDebug" tofile="${lsp.build.dir}/java/maven/bin/mvnDebug.sh"/>
117+
<move file="${lsp.build.dir}/java/maven/bin/mvnyjp" tofile="${lsp.build.dir}/java/maven/bin/mvnyjp.sh"/>
113118
<move file="${lsp.build.dir}/bin/nbcode" tofile="${lsp.build.dir}/bin/nbcode.sh" />
114119
<move file="${lsp.build.dir}/platform/lib/nbexec" tofile="${lsp.build.dir}/platform/lib/nbexec.sh" />
115120
<replace file="${lsp.build.dir}/bin/nbcode.sh" token="/platform*/lib/nbexec" value="/platform*/lib/nbexec.sh"/>
@@ -118,7 +123,11 @@
118123
<replace file="${lsp.build.dir}/platform/lib/nbexec64.dll" token="java.security.manager" value="no.java.secur.manager" encoding="ISO-8859-1"/>
119124
<chmod file="${lsp.build.dir}/bin/nbcode.sh" perm="u+x" />
120125
<chmod file="${lsp.build.dir}/platform/lib/nbexec.sh" perm="u+x"/>
121-
<chmod file="${lsp.build.dir}/java/maven/bin/mvn.sh" perm="u+x" />
126+
<chmod file="${lsp.build.dir}/extide/ant/bin/ant.sh" perm="u+x"/>
127+
<chmod file="${lsp.build.dir}/extide/ant/bin/antRun.sh" perm="u+x"/>
128+
<chmod file="${lsp.build.dir}/java/maven/bin/mvn.sh" perm="u+x"/>
129+
<chmod file="${lsp.build.dir}/java/maven/bin/mvnDebug.sh" perm="u+x"/>
130+
<chmod file="${lsp.build.dir}/java/maven/bin/mvnyjp.sh" perm="u+x"/>
122131
</target>
123132
<target name="add-extra-modules" depends="build-lsp-server,build-l10n-bundles" if="extra.modules">
124133
<ant dir="../../nbbuild" target="build-nbms" inheritall="false" inheritrefs="false">
@@ -238,14 +247,28 @@
238247
<arg value="-Dexec.mainClass=org.netbeans.prepare.bundles.PrepareBundles" />
239248
<arg value="-Dexec.args=${build.dir}/bundles ${nb_all}" />
240249
</exec>
241-
250+
242251
<mkdir dir="${build.dir}/vsce" />
243252
<exec executable="npm${cmd.suffix}" failonerror="true" dir="${build.dir}/vsce">
244253
<arg value="install" />
245254
<arg value="--save" />
246255
<arg value="@vscode/[email protected]" />
247256
</exec>
248-
257+
258+
<move todir="${basedir}/vscode/node_modules" includeemptydirs="false">
259+
<fileset dir="${basedir}/vscode/node_modules">
260+
<include name="**/LICENSE"/>
261+
<include name="**/NOTICE"/>
262+
</fileset>
263+
<mapper type="glob" from="*" to="*.txt"/>
264+
</move>
265+
<move todir="${basedir}/vscode/nbcode" includeemptydirs="false">
266+
<fileset dir="${basedir}/vscode/nbcode">
267+
<include name="**/LICENSE"/>
268+
<include name="**/NOTICE"/>
269+
</fileset>
270+
<mapper type="glob" from="*" to="*.txt"/>
271+
</move>
249272
<copy file="${basedir}/LICENSE.txt" todir="${basedir}/vscode" />
250273
<copy file="${basedir}/THIRD_PARTY_LICENSES.txt" todir="${basedir}/vscode" />
251274
<copy file="${basedir}/README.md" todir="${basedir}/vscode" />

patches/project-marker-jdk.diff

Lines changed: 17 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,17 @@
1+
diff --git a/java/java.openjdk.project/licenseinfo.xml b/java/java.openjdk.project/licenseinfo.xml
2+
index a2c0157ee9..4cb7888b23 100644
3+
--- a/java/java.openjdk.project/licenseinfo.xml
4+
+++ b/java/java.openjdk.project/licenseinfo.xml
5+
@@ -27,7 +27,7 @@
6+
<file>src/org/netbeans/modules/java/openjdk/project/resources/jdk-project.png</file> <!-- copy of: java.j2seproject/src/org/netbeans/modules/java/j2seproject/ui/resources/j2seProject.png-->
7+
<file>src/org/netbeans/modules/java/openjdk/project/resources/nativeFilesFolderOpened.gif</file> <!-- copy of: openide.loaders/src/org/openide/loaders/defaultFolderOpen.gif-->
8+
<file>src/org/netbeans/modules/java/openjdk/project/resources/nativeFilesFolder.gif</file> <!-- copy of: openide.loaders/src/org/openide/loaders/defaultFolder.gif-->
9+
- <file>release/patterns/project-marker-jdk</file>
10+
+ <file>release/patterns/project-marker-jdk.txt</file>
11+
<license ref="Apache-2.0-ASF" />
12+
<comment type="COMMENT_UNSUPPORTED" />
13+
</fileset>
14+
diff --git a/java/java.openjdk.project/release/patterns/project-marker-jdk b/java/java.openjdk.project/release/patterns/project-marker-jdk.txt
15+
similarity index 100%
16+
rename from java/java.openjdk.project/release/patterns/project-marker-jdk
17+
rename to java/java.openjdk.project/release/patterns/project-marker-jdk.txt

vscode/.vscodeignore

Lines changed: 3 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -12,4 +12,6 @@ vsc-extension-quickstart.md
1212
coverage/**
1313
**/.nycrc.json
1414
esbuild.js
15-
images/*
15+
images/*
16+
**/.lastModified
17+
**/.npmignore

0 commit comments

Comments
 (0)