Project: fix copy/paste bug, found by FindBugs

Project.java:2512, DLS_DEAD_LOCAL_STORE_SHADOWS_FIELD, Priority: High

DLS: Dead store to target rather than field with same name in
aQute.bnd.build.Project.getCommonJavac(boolean)

This instruction assigns a value to a local variable, but the value is
not read or used in any subsequent instruction. Often, this indicates
an error, because the value computed is never used. There is a field with
the same name as the local variable. Did you mean to assign to that
variable instead?

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Fix invalid format string, found by Findbugs

CachingUriResourceHandle.java:196
VA_FORMAT_STRING_EXTRA_ARGUMENTS_PASSED, Priority: High

FS: Format-string method String.format(String, Object[]) called with
format string "Invalid SHA on remote resource" wants 0 arguments but is
given 1 in aQute.bnd.deployer.repository.CachingUriResourceHandle.request()

Builder.java:1633
VA_FORMAT_STRING_EXTRA_ARGUMENTS_PASSED, Priority: High

FS: Format-string method String.format(String, Object[]) called with
format string "${githead}, provide the SHA for the current git head" wants
0 arguments but is given 1 in aQute.bnd.osgi.Builder._githead(String[])

A format-string method with a variable number of arguments is called, but
more arguments are passed than are actually used by the format string.
This won't cause a runtime exception, but the code may be silently
omitting information that was intended to be included in the formatted
string.

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Fix 2 'dead local store' bugs found by findbugs

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Fix 2 'unrelated types' bugs found by findbugs

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>
Fix 2 'unrelated types' bugs found by findbugs

Signed-off-by: Ferry Huberts <ferry.huberts@pelagic.nl>