Skip to content

Commit 18e8178

Browse files
committed
hint that Eclipse restart is required
adjust description of the warning, so at least it is mentioned somewhere. unfortunatelly I could not figure out how to reliably sync the Processor.pendatic field with the .bnd files on change. The problem is that Processor.pendatic can be set via .bnd files but also via Processor.setPedantic(boolean) e.g. from testcases, there is some magic caching going on in Processor.begin() and Processor.getProperties() which I was not able to figure out, what to call so that the Workspace Refresh Button behaves the same as an Eclipse restart. Signed-off-by: Christoph Rueger <[email protected]> Update pedantic.md
1 parent eab7011 commit 18e8178

File tree

2 files changed

+6
-5
lines changed

2 files changed

+6
-5
lines changed

biz.aQute.bndlib/src/aQute/bnd/help/Syntax.java

+3-1
Original file line numberDiff line numberDiff line change
@@ -576,7 +576,9 @@ null, null, new Syntax("name", "The display name of the developer", "name='Peter
576576
"If set, is used a template to calculate the output file. It can use any macro but the ${@bsn} and ${@version} macros refer to the current JAR being saved. The default is bsn + \".jar\".",
577577
OUTPUTMASK + "=my_file.zip", null, null),
578578
new Syntax(PACKAGEINFOTYPE, "Sets the different types of package info.", PACKAGEINFOTYPE + "=osgi", null, null),
579-
new Syntax(PEDANTIC, "Warn about things that are not really wrong but still not right.", PEDANTIC + "=true",
579+
new Syntax(PEDANTIC,
580+
"Warn about things that are not really wrong but still not right. Note: Currently a change to this instruction requires a restart of Eclipse.",
581+
PEDANTIC + "=true",
580582
"true,false", Verifier.TRUEORFALSEPATTERN),
581583

582584
new Syntax(PLUGIN, "Define the plugins.",

docs/_instructions/pedantic.md

+3-4
Original file line numberDiff line numberDiff line change
@@ -5,8 +5,7 @@ title: -pedantic BOOLEAN
55
summary: Warn about things that are not really wrong but still not right.
66
---
77

8-
protected void begin() {
9-
if (isTrue(getProperty(PEDANTIC)))
10-
setPedantic(true);
11-
}
8+
When setting this instruction to `true` there will be more warnings about *things that are not really wrong but still not right*.
9+
It can be helpful to fix problems in your workspace.
1210

11+
**Note for Eclipse users:** Currently a change to this instruction requires a restart of Eclipse. The Refresh-Workspace button is not enough.

0 commit comments

Comments
 (0)