Merge branch 'stable-5.11' into stable-5.12
* stable-5.11: Add missing @since tags Add missing @since tag Add missing @since tags Remove unused import in ApacheSshTest Update maven plugins Ignore missing javadoc in test bundles storage: file: De-duplicate File.exists()+File.isFile() RefDirectory.scanRef: Re-use file existence check done in snapshot creation FileSnapshot: Lazy load file store attributes cache Update eclipse-jarsigner-plugin to 1.3.2 Fix p2 repository URLs Change-Id: I72f39fbe82dd578d71aa2e3ceccebb39a25c4e00
This commit is contained in:
commit
1cffba4385
|
@ -125,6 +125,7 @@ public void setUploadPackFactory(UploadPackFactory<HttpServletRequest> f) {
|
|||
*
|
||||
* @param h
|
||||
* A custom error handler for git-upload-pack.
|
||||
* @since 5.6
|
||||
*/
|
||||
public void setUploadPackErrorHandler(UploadPackErrorHandler h) {
|
||||
assertNotInitialized();
|
||||
|
@ -163,6 +164,7 @@ public void setReceivePackFactory(ReceivePackFactory<HttpServletRequest> f) {
|
|||
*
|
||||
* @param h
|
||||
* A custom error handler for git-receive-pack.
|
||||
* @since 5.7
|
||||
*/
|
||||
public void setReceivePackErrorHandler(ReceivePackErrorHandler h) {
|
||||
assertNotInitialized();
|
||||
|
|
|
@ -31,7 +31,7 @@
|
|||
* If a custom handler is not specified, JGit will use the default error
|
||||
* handler.
|
||||
*
|
||||
* @since 5.6
|
||||
* @since 5.7
|
||||
*/
|
||||
public interface ReceivePackErrorHandler {
|
||||
/**
|
||||
|
|
|
@ -80,6 +80,7 @@ public String toString() {
|
|||
* @param userGitConfig
|
||||
* set another user-level git config
|
||||
* @return the old user-level git config
|
||||
* @since 5.1.9
|
||||
*/
|
||||
public FileBasedConfig setUserGitConfig(FileBasedConfig userGitConfig) {
|
||||
FileBasedConfig old = this.userGitConfig;
|
||||
|
@ -92,6 +93,7 @@ public FileBasedConfig setUserGitConfig(FileBasedConfig userGitConfig) {
|
|||
*
|
||||
* @param jgitConfig
|
||||
* set the jgit configuration
|
||||
* @since 5.5
|
||||
*/
|
||||
public void setJGitConfig(FileBasedConfig jgitConfig) {
|
||||
this.jgitConfig = jgitConfig;
|
||||
|
@ -103,6 +105,7 @@ public void setJGitConfig(FileBasedConfig jgitConfig) {
|
|||
* @param systemGitConfig
|
||||
* the new system-level git config
|
||||
* @return the old system-level config
|
||||
* @since 5.1.9
|
||||
*/
|
||||
public FileBasedConfig setSystemGitConfig(FileBasedConfig systemGitConfig) {
|
||||
FileBasedConfig old = this.systemGitConfig;
|
||||
|
|
|
@ -60,6 +60,7 @@ public static class RepeatedTestException extends RuntimeException {
|
|||
*
|
||||
* @param message
|
||||
* the error message
|
||||
* @since 5.1.9
|
||||
*/
|
||||
public RepeatedTestException(String message) {
|
||||
super(message);
|
||||
|
|
|
@ -340,6 +340,7 @@ public static String slashify(String str) {
|
|||
* greater than then the lastmodification time of lastfile.
|
||||
* @throws InterruptedException
|
||||
* @throws IOException
|
||||
* @since 5.1.9
|
||||
*/
|
||||
public static Instant fsTick(File lastFile)
|
||||
throws InterruptedException,
|
||||
|
|
|
@ -21,6 +21,8 @@
|
|||
* This class is used when it's required to load jgit classes in separate
|
||||
* classloader for each test class. It can be needed to isolate static field
|
||||
* initialization between separate tests.
|
||||
*
|
||||
* @since 5.5
|
||||
*/
|
||||
public class SeparateClassloaderTestRunner extends BlockJUnit4ClassRunner {
|
||||
|
||||
|
|
|
@ -339,6 +339,7 @@ public RevObject get(RevTree tree, String path)
|
|||
* zero or more IDs of the commit's parents.
|
||||
* @return the ID of the new commit.
|
||||
* @throws Exception
|
||||
* @since 5.5
|
||||
*/
|
||||
public ObjectId unparsedCommit(ObjectId... parents) throws Exception {
|
||||
return unparsedCommit(1, tree(), parents);
|
||||
|
@ -431,6 +432,7 @@ public RevCommit commit(final int secDelta, final RevTree tree,
|
|||
* zero or more IDs of the commit's parents.
|
||||
* @return the ID of the new commit.
|
||||
* @throws Exception
|
||||
* @since 5.5
|
||||
*/
|
||||
public ObjectId unparsedCommit(final int secDelta, final RevTree tree,
|
||||
final ObjectId... parents) throws Exception {
|
||||
|
|
|
@ -20,6 +20,8 @@
|
|||
|
||||
/**
|
||||
* Utility methods for handling timestamps
|
||||
*
|
||||
* @since 5.1.9
|
||||
*/
|
||||
public class TimeUtil {
|
||||
/**
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.10" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.10" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -6,3 +6,4 @@ include "orbit/R20210602031627-2021-06.tpd"
|
|||
location "https://download.eclipse.org/releases/2018-12/" {
|
||||
org.eclipse.osgi lazy
|
||||
}
|
||||
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.11" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.11" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.12" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.12" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.13" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.13" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.14" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.14" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -6,3 +6,4 @@ include "orbit/R20210602031627-2021-06.tpd"
|
|||
location "https://download.eclipse.org/releases/2019-12/201912181000/" {
|
||||
org.eclipse.osgi lazy
|
||||
}
|
||||
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.15" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.15" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.16" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.16" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.17" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.17" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.18" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.18" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.19-staging" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.19-staging" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.20" sequenceNumber="1635977559">
|
||||
<target name="jgit-4.20" sequenceNumber="1638646959">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.6" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.6" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.7" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.7" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.8" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.8" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -1,7 +1,7 @@
|
|||
<?xml version="1.0" encoding="UTF-8" standalone="no"?>
|
||||
<?pde?>
|
||||
<!-- generated with https://github.com/eclipse-cbi/targetplatform-dsl -->
|
||||
<target name="jgit-4.9" sequenceNumber="1635977558">
|
||||
<target name="jgit-4.9" sequenceNumber="1638646958">
|
||||
<locations>
|
||||
<location includeMode="slicer" includeAllPlatforms="false" includeSource="true" includeConfigurePhase="true" type="InstallableUnit">
|
||||
<unit id="org.eclipse.jetty.client" version="9.4.41.v20210516"/>
|
||||
|
|
|
@ -186,6 +186,12 @@ public static FileSnapshot save(Instant modified) {
|
|||
/** measured FileStore attributes */
|
||||
private FileStoreAttributes fileStoreAttributeCache;
|
||||
|
||||
/**
|
||||
* if {@code true} read filesystem time resolution from configuration file
|
||||
* otherwise use fallback resolution
|
||||
*/
|
||||
private boolean useConfig;
|
||||
|
||||
/**
|
||||
* Object that uniquely identifies the given file, or {@code
|
||||
* null} if a file key is not available
|
||||
|
@ -222,9 +228,7 @@ protected FileSnapshot(File file) {
|
|||
protected FileSnapshot(File file, boolean useConfig) {
|
||||
this.file = file;
|
||||
this.lastRead = Instant.now();
|
||||
this.fileStoreAttributeCache = useConfig
|
||||
? FS.getFileStoreAttributes(file.toPath())
|
||||
: FALLBACK_FILESTORE_ATTRIBUTES;
|
||||
this.useConfig = useConfig;
|
||||
BasicFileAttributes fileAttributes = null;
|
||||
try {
|
||||
fileAttributes = FS.DETECTED.fileAttributes(file);
|
||||
|
@ -379,7 +383,7 @@ public void setClean(FileSnapshot other) {
|
|||
* if sleep was interrupted
|
||||
*/
|
||||
public void waitUntilNotRacy() throws InterruptedException {
|
||||
long timestampResolution = fileStoreAttributeCache
|
||||
long timestampResolution = fileStoreAttributeCache()
|
||||
.getFsTimestampResolution().toNanos();
|
||||
while (isRacyClean(Instant.now())) {
|
||||
TimeUnit.NANOSECONDS.sleep(timestampResolution);
|
||||
|
@ -416,6 +420,15 @@ public boolean equals(Object obj) {
|
|||
return equals(other);
|
||||
}
|
||||
|
||||
/**
|
||||
* Check if the file exists
|
||||
*
|
||||
* @return true if the file exists
|
||||
*/
|
||||
public boolean fileExists() {
|
||||
return !MISSING_FILEKEY.equals(this.fileKey);
|
||||
}
|
||||
|
||||
/** {@inheritDoc} */
|
||||
@Override
|
||||
public int hashCode() {
|
||||
|
@ -500,10 +513,10 @@ private boolean isRacyClean(Instant read) {
|
|||
}
|
||||
|
||||
private long getEffectiveRacyThreshold() {
|
||||
long timestampResolution = fileStoreAttributeCache
|
||||
long timestampResolution = fileStoreAttributeCache()
|
||||
.getFsTimestampResolution().toNanos();
|
||||
long minRacyInterval = fileStoreAttributeCache.getMinimalRacyInterval()
|
||||
.toNanos();
|
||||
long minRacyInterval = fileStoreAttributeCache()
|
||||
.getMinimalRacyInterval().toNanos();
|
||||
long max = Math.max(timestampResolution, minRacyInterval);
|
||||
// safety margin: factor 2.5 below 100ms otherwise 1.25
|
||||
return max < 100_000_000L ? max * 5 / 2 : max * 5 / 4;
|
||||
|
@ -563,4 +576,13 @@ private boolean isSizeChanged(long currSize) {
|
|||
}
|
||||
return changed;
|
||||
}
|
||||
|
||||
private FileStoreAttributes fileStoreAttributeCache() {
|
||||
if (fileStoreAttributeCache == null) {
|
||||
fileStoreAttributeCache = useConfig
|
||||
? FS.getFileStoreAttributes(file.toPath().getParent())
|
||||
: FALLBACK_FILESTORE_ATTRIBUTES;
|
||||
}
|
||||
return fileStoreAttributeCache;
|
||||
}
|
||||
}
|
||||
|
|
|
@ -152,7 +152,7 @@ public long getPackSize() {
|
|||
String p = pack.getAbsolutePath();
|
||||
String i = p.substring(0, p.length() - ".pack".length()) + ".idx"; //$NON-NLS-1$ //$NON-NLS-2$
|
||||
File idx = new File(i);
|
||||
if (idx.exists() && idx.isFile())
|
||||
if (idx.isFile())
|
||||
size += idx.length();
|
||||
return size;
|
||||
}
|
||||
|
|
|
@ -1092,10 +1092,14 @@ LooseRef scanRef(LooseRef ref, String name) throws IOException {
|
|||
final int limit = 4096;
|
||||
final byte[] buf;
|
||||
FileSnapshot otherSnapshot = FileSnapshot.save(path);
|
||||
if (!otherSnapshot.fileExists()) {
|
||||
return null;
|
||||
}
|
||||
|
||||
try {
|
||||
buf = IO.readSome(path, limit);
|
||||
} catch (FileNotFoundException noFile) {
|
||||
if (path.exists() && path.isFile()) {
|
||||
if (path.isFile()) {
|
||||
throw noFile;
|
||||
}
|
||||
return null; // doesn't exist or no file; not a reference.
|
||||
|
|
14
pom.xml
14
pom.xml
|
@ -168,13 +168,13 @@
|
|||
<httpcore-version>4.4.14</httpcore-version>
|
||||
<slf4j-version>1.7.30</slf4j-version>
|
||||
<log4j-version>1.2.15</log4j-version>
|
||||
<maven-javadoc-plugin-version>3.3.0</maven-javadoc-plugin-version>
|
||||
<maven-javadoc-plugin-version>3.3.1</maven-javadoc-plugin-version>
|
||||
<tycho-extras-version>1.7.0</tycho-extras-version>
|
||||
<gson-version>2.8.6</gson-version>
|
||||
<bouncycastle-version>1.65</bouncycastle-version>
|
||||
<spotbugs-maven-plugin-version>4.2.3</spotbugs-maven-plugin-version>
|
||||
<spotbugs-maven-plugin-version>4.3.0</spotbugs-maven-plugin-version>
|
||||
<maven-project-info-reports-plugin-version>3.1.2</maven-project-info-reports-plugin-version>
|
||||
<maven-jxr-plugin-version>3.0.0</maven-jxr-plugin-version>
|
||||
<maven-jxr-plugin-version>3.1.1</maven-jxr-plugin-version>
|
||||
<maven-surefire-plugin-version>3.0.0-M5</maven-surefire-plugin-version>
|
||||
<maven-surefire-report-plugin-version>${maven-surefire-plugin-version}</maven-surefire-report-plugin-version>
|
||||
<maven-compiler-plugin-version>3.8.1</maven-compiler-plugin-version>
|
||||
|
@ -255,7 +255,7 @@
|
|||
<plugin>
|
||||
<groupId>org.apache.maven.plugins</groupId>
|
||||
<artifactId>maven-dependency-plugin</artifactId>
|
||||
<version>3.1.2</version>
|
||||
<version>3.2.0</version>
|
||||
</plugin>
|
||||
|
||||
<plugin>
|
||||
|
@ -307,7 +307,7 @@
|
|||
<plugin>
|
||||
<groupId>org.apache.maven.plugins</groupId>
|
||||
<artifactId>maven-pmd-plugin</artifactId>
|
||||
<version>3.14.0</version>
|
||||
<version>3.15.0</version>
|
||||
<configuration>
|
||||
<sourceEncoding>utf-8</sourceEncoding>
|
||||
<minimumTokens>100</minimumTokens>
|
||||
|
@ -345,7 +345,7 @@
|
|||
<plugin>
|
||||
<groupId>org.jacoco</groupId>
|
||||
<artifactId>jacoco-maven-plugin</artifactId>
|
||||
<version>0.8.6</version>
|
||||
<version>0.8.7</version>
|
||||
</plugin>
|
||||
<plugin>
|
||||
<groupId>org.apache.maven.plugins</groupId>
|
||||
|
@ -411,7 +411,7 @@
|
|||
<plugin>
|
||||
<groupId>org.apache.maven.plugins</groupId>
|
||||
<artifactId>maven-enforcer-plugin</artifactId>
|
||||
<version>3.0.0-M3</version>
|
||||
<version>3.0.0</version>
|
||||
<executions>
|
||||
<execution>
|
||||
<id>enforce-maven</id>
|
||||
|
|
Loading…
Reference in New Issue