aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
...
* | | | | | | | | DfsReader/PackFile: Implement isNotLargerThan using the obj size idxIvan Frade2023-07-134-0/+351
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | isNotLargerThan() can avoid reading the size of a blob from disk using the object size idx if available. Load the object size index in the DfsPackfile following the same pattern than the other indices. Override isNotLargerThan in DfsReader to use the index when available. Following CL introduces the writing of the object size index and the tests cover this code. Change-Id: I15c95b84c1424707c487a7d29c5c46b1a9d0ceba
* | | | | | | | | [releng] Use tycho 4.0.0 instead of 4.0.0-SNAPSHOTThomas Wolf2023-07-111-5/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Now that it is released there is no need anymore to use a snapshot version. Change-Id: Idd35c48022370abf18049ef4b6ddd6253613888e Signed-off-by: Thomas Wolf <twolf@apache.org>
* | | | | | | | | Update mockito to 5.4.0 and bytebuddy to 1.14.5Matthias Sohn2023-07-0217-74/+182
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia9de3f9fb6f51ac55a7c551cab4ce199318c1114
* | | | | | | | | Merge branch 'stable-6.6'Matthias Sohn2023-07-0127-44/+44
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.6: Update Orbit to R20230531010532 for 2023-06 Bazel: Fix remote build execution for Java 17 Bump bazel vesion to 6.2.0 Change-Id: I107eb2cd1ce3cb7670e7418ffd74a7b94ab858a6
| * | | | | | | | Update Orbit to R20230531010532 for 2023-06Matthias Sohn2023-06-0925-38/+38
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I844efc4bec153931f0a7b3c694bade4f5b166295
| * | | | | | | | Merge "Merge branch 'stable-6.5' into stable-6.6" into stable-6.6Matthias Sohn2023-06-080-0/+0
| |\ \ \ \ \ \ \ \
| * | | | | | | | | Bazel: Fix remote build execution for Java 17David Ostrovsky2023-05-312-6/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bazel build and test with remote17 configuration was actually executed locally, and not remotely. Rename remote configuration setting for Java 11 from remote to remote11 and derive the common remote configuration settings for remote11 and remote17 configurations. Also remove deprecated --host_javabase and --javabase options. Test Plan: Verify that the remote build for Java 17 is actually executed remotely: $ bazel test --config=remote17 --remote_instance_name=$PROJECT_NAME \ org.eclipse.jgit.test/... Change-Id: I10ad8c1d28c92fed9e465c2d36770ed4fb2940c3
| * | | | | | | | | Bump bazel vesion to 6.2.0David Ostrovsky2023-05-313-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also demote severity level for bug pattern: PreferredInterfaceType, aka MutableConstantField: [1] from error to warning. Also fix DoubleBraceInitialization error prone bug pattern [2]. [1] https://errorprone.info/bugpattern/PreferredInterfaceType [2] https://errorprone.info/bugpattern/DoubleBraceInitialization Change-Id: I04b7d0792e67cbc51e3939bd47c2ea13a685cf0d (cherry picked from commit 0f071a7bcee1f551652b2ae017f3e0e134c7eb74)
* | | | | | | | | | Fix S3Repository getSize to handle larger object sizesqin shulei2023-06-291-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Update `getSize` method in `S3Repository` to handle larger object sizes. The method previously used `Integer.parseInt` to parse the `Content-Length` header of an HTTP response, which limited the maximum object size to 2 GB. Replaces `Integer.parseInt` with `Long.parseLong`, allowing the method to handle object sizes larger than 2 GB. - Use minio as local S3 service for gerrit lfs plugin - The minio seems will return the Content-length Change-Id: Ia3a5fd1a335643786714aff3fcc7d10a6b152058 Signed-off-by: qin shulei <qinsl0106@thundersoft.com>
* | | | | | | | | | DfsPackFile: make #getReverseIdx publicAnna Papitto2023-06-271-1/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The DfsPackFile#getReverseIdx method, which wraps creating a PackReverseIndex in caching, was package-private. This caused implementations on top of DfsPackFile to directly instantiate a PackReverseIndex in cases where it would benefit from caching. Instead, make #getReverseIdx public so that the caching logic can be reused by implementations where appropriate. Change-Id: I4553e514a4ac320bfe2455c00023343ad97f9d15 Signed-off-by: Anna Papitto <annapapitto@google.com>
* | | | | | | | | | Add missing bazel dependency to benchmarksMatthias Sohn2023-06-271-0/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I330cbdbe7fdd2e3523cfc555adb15b577e858083
* | | | | | | | | | Update JMH to 1.36Matthias Sohn2023-06-272-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ie5cbbbae59fac9b0044808bd098d47f530126c67
* | | | | | | | | | Update bouncycastle to 1.75Matthias Sohn2023-06-2715-74/+74
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I7538759005b9a4eb8f1ae9337ce0056500eb7227
* | | | | | | | | | Update org.eclipse.jdt:ecj to 3.34.0Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I59715e1abfdefdf883290557806a9a1cbb34c3b3
* | | | | | | | | | Update eclipse-jarsigner-plugin to 1.4.2Matthias Sohn2023-06-272-2/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I3f92a32ccf795ae8c6c4e1699d0040ac84d743c2
* | | | | | | | | | Update maven-surefire-report-plugin to 3.0.0Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id1f33c85045d280ac7238a9f1ba63a7c19f0e455
* | | | | | | | | | Update maven-project-info-reports-plugin to 3.4.3Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I1bd92105af3cc8eb2e6116677e50365702b58f5c
* | | | | | | | | | Update maven-compiler-plugin to 3.11.0Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Ia63fd770ebb8b44dcb3e09721c890fc2b42cbcec
* | | | | | | | | | Update maven-enforcer-plugin to 3.3.0Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9ad8bf71a492017ba3d71b527acd02ad22612fb9
* | | | | | | | | | Update build-helper-maven-plugin to 3.4.0Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9b5e8a6b0baebd67d84f48d95b83c6c802829aea
* | | | | | | | | | Update spring-boot-maven-plugin to 2.7.13Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I9d28540971a9c45f961ccfef1123a0a4ba3f88f2
* | | | | | | | | | Update maven-source-plugin to 3.3.0Matthias Sohn2023-06-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4735538aeeb92a01009aad1ca837cb636dcd4b93
* | | | | | | | | | PackReverseIndex: separate out the computed implementationAnna Papitto2023-06-216-150/+229
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PackReverseIndex is a concrete class whose implementation is computed from a pack's forward index. Callers which have a reverse index file may want to use an implementation that is file-based instead. Generalize PackReverseIndex into an interface without implementation-specific logic and separate out the logic for the computed implementation into a new concrete class. Change-Id: I98d9835363c5e1c8c3c11a81b0761af3cdeaa41a Signed-off-by: Anna Papitto <annapapitto@google.com>
* | | | | | | | | | Default for global (user) git ignore fileThomas Wolf2023-06-197-45/+193
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | C git has a default for git config core.excludesfile: "Its default value is $XDG_CONFIG_HOME/git/ignore. If $XDG_CONFIG_HOME is either not set or empty, $HOME/.config/git/ignore is used instead." [1] Implement this in the WorkingTreeIterator$RootIgnoreNode. To make this testable, mock the "user.home" directory for all JGit tests, otherwise tests might pick up a real user's git ignore file. Also ensure that JGit code always reads "user.home" via the SystemReader. Add tests for both locations. [1] https://git-scm.com/docs/gitignore#_description Bug: 436127 Change-Id: Ie510259320286c3c13a6464a37da1bd9ca1e373a Signed-off-by: Thomas Wolf <twolf@apache.org>
* | | | | | | | | | Fix all Javadoc warnings and fail on themAntoine Musso2023-06-16627-1653/+1204
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This fixes all the javadoc warnings, stops ignoring doclint 'missing' category and fails the build on javadoc warnings for public and protected classes and class members. Since javadoc doesn't allow access specifiers when specifying doclint configuration we cannot set `-Xdoclint:all,-missing/private` hence there is no simple way to skip private elements from doclint. Therefore we check javadoc using the Eclipse Java compiler (which is used by default) and javadoc configuration in `.settings/org.eclipse.jdt.core.prefs` files. This allows more fine grained configuration. We can reconsider this when javadoc starts supporting access specifiers in the doclint configuration. Below are detailled explanations for most modifications. @inheritDoc =========== doclint complains about explicits `{@inheritDoc}` when the parent does not have any documentation. As far as I can tell, javadoc defaults to inherit comments and should only be used when one wants to append extra documentation from the parent. Given the parent has no documentation, remove those usages which doclint complains about. In some case I have moved up the documentation from the concrete class up to the abstract class. Remove `{@inheritDoc}` on overriden methods which don't add additional documentation since javadoc defaults to inherit javadoc of overridden methods. @value to @link =============== In PackConfig, DEFAULT_SEARCH_FOR_REUSE_TIMEOUT and similar are forged from Integer.MAX_VALUE and are thus not considered constants (I guess cause the value would depends on the platform). Replace it with a link to `Integer.MAX_VALUE`. In `StringUtils.toBoolean`, @value was used to refer to the `stringValue` parameter. I have replaced it with `{@code stringValue}`. {@link <url>} to <a> ==================== @link does not support being given an external URL. Replaces them with HTML `<a>`. @since: being invalid ===================== org.eclipse.jgit/src/org/eclipse/jgit/util/Equality.java has an invalid tag `@since: ` due to the extra `:`. Javadoc does not complain about it with version 11.0.18+10 but does with 11.0.19.7. It is invalid regardless. invalid HTML syntax =================== - javadoc doesn't allow <br/>, <p/> and </p> anymore, use <br> and <p> instead - replace <tt>code</tt> by {@code code} - <table> tags don't allow summary attribute, specify caption as <caption>caption</caption> to fix this doclint visibility issue ======================== In the private abstract classes `BaseDirCacheEditor` and `BasePackConnection` links to other methods in the abstract class are inherited in the public subclasses but doclint gets confused and considers them unreachable. The HTML documentation for the sub classes shows the relative links in the sub classes, so it is all correct. It must be a bug somewhere in javadoc. Mute those warnings with: @SuppressWarnings("doclint:missing") Misc ==== Replace `<` and `>` with HTML encoded entities (`&lt; and `&gt;`). In `SshConstants` I went enclosing a serie of -> arrows in @literal. Additional tags =============== Configure maven-javad0c-plugin to allow the following additional tags defined in https://openjdk.org/jeps/8068562: - apiNote - implSpec - implNote Missing javadoc =============== Add missing @params and descriptions Change-Id: I840056389aa59135cfb360da0d5e40463ce35bd0 Also-By: Matthias Sohn <matthias.sohn@sap.com>
* | | | | | | | | | Mark COMMIT_GENERATION_* constants finalAntoine Musso2023-06-092-2/+16
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In org.eclipse.jgit.lib.Constants the constants are all marked final with the exception of: - COMMIT_GENERATION_UNKOWN - COMMIT_GENERATION_NOT_COMPUTED They were introduced by cf70e7cbe4f without the `final` keyword while other constants have it which certainly has been forgotten. The javadoc `{@value}` tag causes raises a warning about the fields not being constants which is how I have discovered the ommission. Change-Id: I0ad87f42355440c7d50158e773a280a0526e9671
* | | | | | | | | | Merge branch 'stable-6.6'Luca Milanesio2023-06-080-0/+0
|\ \ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.6: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: I6011c20b1aca179126393b0e03fbfb0ba1b6d64c Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| * | | | | | | | | Merge branch 'stable-6.5' into stable-6.6Luca Milanesio2023-06-080-0/+0
| |\ \ \ \ \ \ \ \ \ | | |/ / / / / / / / | |/| / / / / / / / | | |/ / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.5: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: Ia2f6f41f27ab430c31871a2cf2cd99dfcbfd9b61 Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | * | | | | | | Merge branch 'stable-6.4' into stable-6.5Luca Milanesio2023-06-081-16/+1
| | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.4: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: I7d922a92b7674723cbf6a93fb7c9bc5c0cdb8206 Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | | * | | | | | Merge branch 'stable-6.3' into stable-6.4Luca Milanesio2023-06-081-16/+1
| | | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.3: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: I33049e70595f097a66e8f4a63b3d8d1c147e878e Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | | | * | | | | Merge branch 'stable-6.2' into stable-6.3Luca Milanesio2023-06-081-16/+1
| | | | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.2: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: I70db1bc8529eb6a66610946946da5447a578bffa Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | | | | * | | | Merge branch 'stable-6.1' into stable-6.2Luca Milanesio2023-06-081-16/+1
| | | | | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.1: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: I1a98e293ef10917b2d8ad64e88be9e82c7bcf693 Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | | | | | * | | Merge branch 'stable-6.0' into stable-6.1Luca Milanesio2023-06-081-16/+1
| | | | | | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.0: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: Idc0d1f8ab4524868b7e9754799f70acc1d24f2cb Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | | | | | | * | Merge branch 'stable-5.13' into stable-6.0Luca Milanesio2023-06-081-16/+1
| | | | | | | |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-5.13: Revert "RefDirectory: Throw exception if CAS of packed ref list fails" Change-Id: I883b21b00317cc6d9951a8a5f9505078ddd2a3a7 Signed-off-by: Luca Milanesio <luca.milanesio@gmail.com>
| | | | | | | | * Revert "RefDirectory: Throw exception if CAS of packed ref list fails"Martin Fick2023-06-071-16/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This reverts commit 9c33f7364d41956240818ba12d8b79d5ea846162. Reason for revert: This change was based on the false claim that the packedrefs file lock is held while the CAS is being done, but it is actually released before the CAS (the in memory lock is still held, however that does not prevent external actors from updating the packedrefs files and then another thread from subsequently re-reading it and updating the in memory packedRefList). Although reverting this change can cause the CAS to fail, it should not actually matter since the failure would indicate that another thread has already updated the in memory packedRefList to either the same version this thread was trying to update it too, or to a more recent version. Either way, failing the CAS is then appropriate and should not be problematic. Although this change reverts the code in the RefDirectory class, it keeps the "improvements" to the test so that it continues to pass reliably. The reason for the quotes around the word "improvements" is because I believe the test alteration actually dramatically changes the intent of the test, and that the original intent of the test is untestable with the GC and RefDirectory classes as is. Bug: 582044 Change-Id: I3acee7527bb542996dcdfaddfb2bdb45ec444db5 Signed-off-by: Martin Fick <quic_mfick@quicinc.com> (cherry picked from commit c5617711a1b4d5d0807cc7eed702b78d114d46b3)
* | | | | | | | | Merge "Merge branch 'stable-6.6'"Matthias Sohn2023-06-070-0/+0
|\ \ \ \ \ \ \ \ \
| * | | | | | | | | Merge branch 'stable-6.6'Matthias Sohn2023-05-310-0/+0
| |\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.6: Prepare 6.6.1-SNAPSHOT builds JGit v6.6.0.202305301015-r Change-Id: I4f7fc395122d4c34ceaa639726e9d0e255641c8d
| | * | | | | | | | Prepare 6.6.1-SNAPSHOT buildsMatthias Sohn2023-05-3091-533/+533
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I0036999e2be076d4ad8231410faeff51bf9cbf52
| | * | | | | | | | JGit v6.6.0.202305301015-rv6.6.0.202305301015-rMatthias Sohn2023-05-3091-125/+125
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthias Sohn <matthias.sohn@sap.com> Change-Id: I33b45b0cf36835b289ecbb5a1a9fc4ad7fc200cd
* | | | | | | | | | PackReverseIndex: use static builder instead of constructorAnna Papitto2023-05-314-6/+18
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | PackReverseIndex instances are created using the constructor directly, which limits control over the construction logic and refactoring opportunities for the class itself. These will be needed for a file-based implementation of the reverse index. Use a static builder method to create a PackReverseIndex instance using a pack's forward index. Change-Id: I4421d907cd61d9ac932df5377e5e28a81679b63f Signed-off-by: Anna Papitto <annapapitto@google.com>
* | | | | | | | | | Gc#writePack: write the reverse index file to diskAnna Papitto2023-05-312-0/+135
|/ / / / / / / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The reverse index is currently created in-memory when needed. A writer for reverse index files was already implemented. Make garbage collection write the reverse index file when the PackConfig enables it. Write it during #writePack, which mirrors how the primary index is written. Change-Id: I50131af6622c41a7b24534aaaf2a423ab4178981 Signed-off-by: Anna Papitto <annapapitto@google.com>
* | | | | | | | | Bump bazel vesion to 6.2.0David Ostrovsky2023-05-293-8/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Also demote severity level for bug pattern: PreferredInterfaceType, aka MutableConstantField: [1] from error to warning. Also fix DoubleBraceInitialization error prone bug pattern [2]. [1] https://errorprone.info/bugpattern/PreferredInterfaceType [2] https://errorprone.info/bugpattern/DoubleBraceInitialization Change-Id: I04b7d0792e67cbc51e3939bd47c2ea13a685cf0d
* | | | | | | | | Merge branch 'stable-6.6'Matthias Sohn2023-05-2525-60/+87
|\| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * stable-6.6: Update to Orbit S20230516204213 Prepare 6.6.0-SNAPSHOT builds JGit v6.6.0.202305241045-m3 Prepare 6.6.0-SNAPSHOT builds JGit v6.6.0.202305031100-m2 Change-Id: Ibceebbce6aebba7a8670de41eb39eb23b14b8c74
| * | | | | | | | Update to Orbit S20230516204213Matthias Sohn2023-05-2425-60/+87
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I4daae47b8d2e244b78dff5ca072e41153e7e6734
| * | | | | | | | Prepare 6.6.0-SNAPSHOT buildsMatthias Sohn2023-05-2491-125/+125
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: If0e4e8ce5f3e2f5170f313fb9b26b4ec0e34dab9
| * | | | | | | | JGit v6.6.0.202305241045-m3v6.6.0.202305241045-m3Matthias Sohn2023-05-2491-125/+125
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthias Sohn <matthias.sohn@sap.com> Change-Id: I204708812b9cb6f98f9c29e28548b91da0d88d91
| * | | | | | | | Merge branch 'master' into stable-6.6Matthias Sohn2023-05-2444-709/+1679
| |\ \ \ \ \ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | * master: GraphObjectIndex: fix search in findGraphPosition Update to Tycho 4.0.0-SNAPSHOT PGP sign p2 artefacts Revert 'Use net.i2p.crypto:eddsa directly from Maven Central' Update dash license-tool-plugin to 1.0.2 Also add suppressed exception if unchecked exception occurs in finally Candidate: use "Objects.equals" instead of "==" Use hamcrest 2.2 directly from Maven Central Use commons-logging directly from Maven Central Update jna to 5.13.0 Use bytebuddy directly from Maven Central Use jna directly from Maven Central Use net.i2p.crypto:eddsa directly from Maven Central Use org.tukaani:xz directly from Maven Central Use args4j directly from Maven Central Use gson directly from Maven Central Remove unused $NON-NLS-1$ Remove unused API filters Switch to Apache MINA sshd 2.10.0 [releng] API filter for PackIndex.DEFAULT_WRITE_REVERSE_INDEX PackExt: add a #getTmpExtension method UploadPack: Record negotiation stats on fetchV2 call RewriteGeneratorTest: Introduce test cases for the RewriteGenerator PackWriter: write the PackReverseIndex file Change-Id: I6c7760a32545320862abcdcc8761c9b728e78182
| * | | | | | | | | Prepare 6.6.0-SNAPSHOT buildsMatthias Sohn2023-05-0491-125/+125
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: Id2414c9f30d159c4f65a3c267373f7417c8ca65a
| * | | | | | | | | JGit v6.6.0.202305031100-m2v6.6.0.202305031100-m2Matthias Sohn2023-05-0391-125/+125
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Signed-off-by: Matthias Sohn <matthias.sohn@sap.com> Change-Id: I30077529655918f44479f4abca36aab4d677ea33
* | | | | | | | | | Prepare 6.7.0-SNAPSHOT buildsMatthias Sohn2023-05-2491-533/+533
| |/ / / / / / / / |/| | | | | | | | | | | | | | | | | | | | | | | | | | Change-Id: I50ff7ee31046cfc29a087c8963be3deae24b1c9c