Skip to content

Fix verify libs and licenses for platform#196

Merged
geertjanw merged 4 commits into
apache:masterfrom
jlahoda:fix-verify-libs-and-licenses-for-platform
Oct 27, 2017
Merged

Fix verify libs and licenses for platform#196
geertjanw merged 4 commits into
apache:masterfrom
jlahoda:fix-verify-libs-and-licenses-for-platform

Conversation

@jlahoda
Copy link
Copy Markdown
Contributor

@jlahoda jlahoda commented Oct 26, 2017

Various fixes to resolve failures reported by:
ant -Dcluster.config=platform verify-libs-and-licenses

This includes fixing file names, fixing licenses, fixing headers in license files, etc.

A notable change: deleted apisupport.harness/external/jnlp-servlet.jar (unclear if it is allowed for use); users building a JNLP build of a NetBeans Platform Application will need to supply the file themselves.

@geertjanw
Copy link
Copy Markdown
Member

OK, thanks, merging.

@geertjanw geertjanw merged commit 4ae6d93 into apache:master Oct 27, 2017
Achal1607 pushed a commit to Achal1607/netbeans that referenced this pull request Feb 21, 2025
…he Explorer panel for non-workspace opened Java files

Fixed runConfiguration.ts: `initializeRunConfiguration()` to search for java files in open editors when no workspace is open.

1. Checked for non-workspace opened files by testing the validity of `vscode.workspace.name` AND `vscode.workspace.workspaceFile`.
2. Searched for java extension file names in `vscode.workspace.textDocuments` when no workspace is open.
3. Fixed `RunConfigurationNode.setValue()` to invoke `WorkspaceConfiguration.update()` with `configurationTarget =` :
    - `true`: when no workspace is open i.e. user global target;
    - `null`: otherwise i.e. workspace target.

Fixes apache#196

Signed-off-by: Siddharth Srinivasan <siddharth.srinivasan@oracle.com>
Achal1607 added a commit to Achal1607/netbeans that referenced this pull request Feb 21, 2025
…rkspace

[JAVAVSCODE apache#196] Run Configuration section is unavailable in the Explorer panel for non-workspace opened Java files
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants