summaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* Prepare 1.3.0 buildsMatthias Sohn2011-12-1040-242/+242
| | | | | Change-Id: I7a1ae73783c95041b59f047a7330e62e7f642149 Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
* Merge branch 'stable-1.2'Matthias Sohn2011-12-105-4/+57
|\ | | | | | | | | | | | | | | | | | | | | * stable-1.2: Fix version.sh Throw API exception when MergeCommand hits checkout conflicts Add methods for configuring platform emulation Fix history rendering not to occupy too many lanes Fix History rendering Change-Id: If71cc760423ae2b76c7435ca4830edc1745556de Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
| * Fix version.shMatthias Sohn2011-12-101-1/+1
| | | | | | | | | | Change-Id: Icdf5d9ea3ca62839cbf7de13dfee9682056b7cef Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
| * Throw API exception when MergeCommand hits checkout conflicts Matthias Sohn2011-12-082-3/+24
| | | | | | | | | | | | | | | | | | | | | | When MergeCommand hit checkout conflicts it did throw an internal JGit exception org.eclipse.jgit.errors.CheckoutConflictException instead of org.eclipse.jgit.api.errors.CheckoutConflictException which it declares to throw. Hence translate the internal exception to the exception declared in the API. Bug: 327573 Change-Id: I1efcd93a43ecbf4a40583e0fc9d8d53cffc98cae Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
| * Add methods for configuring platform emulationRobin Rosenberg2011-12-071-0/+30
| | | | | | | | | | | | | | Specifically we support setting system properties for Windows, generic Unix and current test platform. Change-Id: Ib02be417c4915350dfec64fda3face1138552871
| * Fix history rendering not to occupy too many lanesChristian Halstrick2011-12-061-0/+2
| | | | | | | | | | | | | | | | | | | | | | There was a bug in history rendering which caused jgit to use too many lanes in case lanes get repositioned. Looking at commit 90c11cbaeb83ee9b02238cbd2c0e5bcf68068772 in JGit was one example. Vadim Dmitriev found the problem and the solution. Bug: 365460 Change-Id: I6024265b7a593dcfd4fc612d0baf6652a0092ff4 Also-by: Vadim Dmitriev <dmgloss@mail.ru> Signed-off-by: Christian Halstrick <christian.halstrick@sap.com>
| * Fix History renderingChristian Halstrick2011-12-041-30/+42
| | | | | | | | | | | | | | | | | | There was the possibility that during history rendering we draw a lane "trough" a passed commit. Vadim Dmitriev found that out in bug 335818. I added the needed check to that block of code where it was missing. Bug: 335818 Change-Id: Ic944193b2aca55ff3eb0235d46afa60b7896aa0f Signed-off-by: Christian Halstrick <christian.halstrick@sap.com>
* | Merge "Add simple chain implementations of transport hooks and loggers"Shawn Pearce2011-12-094-0/+384
|\ \
| * | Add simple chain implementations of transport hooks and loggersDave Borowitz2011-12-094-0/+384
| | | | | | | | | | | | | | | | | | | | | Allows callers to effectively run multiple hooks and loggers without modifying the UploadPack/ReceivePack interface. Change-Id: I5b388816b63036ffff08ef3a9b857ccb764cb8c4
* | | Merge "Force jgit.sh to use UTF-8 encoding on Mac"Shawn Pearce2011-12-091-0/+1
|\ \ \
| * | | Force jgit.sh to use UTF-8 encoding on MacRobin Rosenberg2011-12-071-0/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | For some obscure reason the platform encodng in Java on Macs is MacRoman. OS X used UTF-8 as the default encoding so set the encoding property in jgit.sh. Change-Id: I08182c2f8512f799178cee70bcc28d6ee2b9c2b0
* | | | Merge changes I9f1842c2,Ie139133bShawn Pearce2011-12-093-4/+13
|\ \ \ \ | | | | | | | | | | | | | | | | | | | | | | | | | * changes: Throw invalid ref exception instead of invalid remote exception Use constant for logs directory
| * | | | Throw invalid ref exception instead of invalid remote exceptionKevin Sawicki2011-12-082-3/+11
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | ReflogCommand command was throwing an incorrect exception type when an IOException was wrapped and rethrown from the underlying ReflogReader. The IOException cause is now provided to the thrown exception as well. Change-Id: I9f1842c2d414d3e9c658843f9b448bc18891748e
| * | | | Use constant for logs directoryKevin Sawicki2011-12-081-1/+2
| | | | | | | | | | | | | | | | | | | | Change-Id: Ie139133bcbe1ca61c85e86b3484f858bc065821f
* | | | | Merge "Add a no-op UploadPackLogger and use it by default"Shawn Pearce2011-12-082-2/+14
|\ \ \ \ \ | |/ / / / |/| | / / | | |/ / | |/| |
| * | | Add a no-op UploadPackLogger and use it by defaultDave Borowitz2011-12-082-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | This matches the behavior of the hooks in UploadPack. Change-Id: I21277e52ee9452237d87084f06bb140dfa6bf608
* | | | Merge "Set expected old object id to all zeros when head id is null"Shawn Pearce2011-12-081-1/+4
|\ \ \ \
| * | | | Set expected old object id to all zeros when head id is nullKevin Sawicki2011-12-061-1/+4
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | This will be the case on the initial commit to a repository and the ref update should expect a non-existent ref Change-Id: Iaa06e47e6e8cc4a0281c7683b367d4806dd980ea
* | | | Merge "Use System.arraycopy to copy parent object id array"Shawn Pearce2011-12-081-2/+1
|\ \ \ \
| * | | | Use System.arraycopy to copy parent object id arrayKevin Sawicki2011-12-071-2/+1
| |/ / / | | | | | | | | | | | | Change-Id: Iee475584f0a65b8b77610cd277f15295b68dfe80
* | | | Merge "[findBugs] Implement Serializable interface in PlotLane"Shawn Pearce2011-12-081-1/+5
|\ \ \ \
| * | | | [findBugs] Implement Serializable interface in PlotLaneKevin Sawicki2011-12-081-1/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This class is referenced by PlotCommit which is Serializable Change-Id: If79b6a593dc53fbb7c4f0a69a9d0dca02a8cb63c
* | | | | Add toString() to HunkHeaderTomasz Zarna2011-12-081-0/+13
|/ / / / | | | | | | | | | | | | | | | | | | | | | | | | Since FileHeader provides toString() method (via DiffEntry) we could add a similar method to HunkHeader. Change-Id: I7886e5b8f775fa8e8478ac5af37d90b6ef677d8b
* / / / Stream left open in WorkingTreeIterator#filterCleanTomasz Zarna2011-12-061-1/+5
|/ / / | | | | | | | | | Change-Id: I17c2709f2613536c02c9ea2977d936b87029ac0d
* | | Merge changes I5381e110,I5534b560Shawn O. Pearce2011-12-055-22/+268
|\ \ \ | | | | | | | | | | | | | | | | | | | | * changes: Discard request HTTP bodies for status code <400 Ensure all smart HTTP errors are sent to clients
| * | | Discard request HTTP bodies for status code <400Shawn O. Pearce2011-12-014-7/+59
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The HTTP RFCs require a server to fully consume the request body before it can return a non-error status code, which is any code below 400. JGit returns most Git level errors inside of an HTTP 200 OK response, and sometimes this happens before the entire request was consumed from the servlet container. In such cases the body must be skipped or read until EOF is reached, ensuring the HTTP keep-alive semantics will work for the next request on the same TCP connection. HTTP status codes >= 400 may be returned without consuming the body, and a servlet container must set "Connection: close" in the response headers when this happens, since the state of the request body is not well defined with an early abort. With the introduction of sendError() in GitSmartHttpTools there are only a handful of locations that need to worry about the request body being consumed, so sprinkle the call in as necessary. Change-Id: I5381e110585f780c01a764df8e27c80aacf5146e
| * | | Ensure all smart HTTP errors are sent to clientsShawn O. Pearce2011-12-013-15/+209
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Error messages are typically short, below the 32 KiB in-memory buffer size of the SmartOutputStream. When an error is queued up for sending to a client and an exception is thrown up into the servlet handler we discarded the message and sent nothing to the client, as the messages were stuck inside of the SmartOutputStream buffer. Hoist the creation of the output stream above the invocation of try block of the service, and use close() in the few catch blocks that assume there are buffered messages ready for transmission. This will ensure errors from unpacking a stream in ReceivePack are sent off to a client correctly, as previously these were causing no status report to arrive at the client side as the data was stuck in the buffer. Change-Id: I5534b560697731121f48979ae077aa7c95b8e39c
* | | | Merge "Always close the GZIPOutputStream to release Deflater"Robin Rosenberg2011-12-021-2/+5
|\| | |
| * | | Always close the GZIPOutputStream to release DeflaterShawn O. Pearce2011-11-301-2/+5
| | |/ | |/| | | | | | | | | | | | | | | | | | | The stream must be closed to ensure the native resources associated with its internal Deflater instance are cleaned up early, instead of waiting for GC to identify the dead object and finialize it. Change-Id: Ic31b5df563f19404ed4682556999f4332aa61562
* | | Merge "Fix History rendering"Shawn Pearce2011-12-021-30/+42
|\ \ \
| * | | Fix History renderingChristian Halstrick2011-12-021-30/+42
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | There was the possibility that during history rendering we draw a lane "trough" a passed commit. Vadim Dmitriev found that out in bug 335818. I added the needed check to that block of code where it was missing. Bug: 335818 Change-Id: Ic944193b2aca55ff3eb0235d46afa60b7896aa0f Signed-off-by: Christian Halstrick <christian.halstrick@sap.com>
* | | Merge "Add percent-encoding of reserved characters in URIish"Shawn Pearce2011-12-022-30/+374
|\ \ \ | |/ / |/| |
| * | Add percent-encoding of reserved characters in URIishRobin Rosenberg2011-12-022-30/+374
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | We do this for the the names that have an explicit scheme and do it both ways. The URIish is parsed before decoding. Only a few special characters are encoded for the path part of the URI, i.e. space, non-ASCII and control characters. The percent encoding is assumed to be a stream encoding so we interpret it as UTF-8. Change-Id: I82d1910df9472e21d7212a2b984ff7d8fb2cbf0f
* | | Fix HTTP unit testsShawn O. Pearce2011-11-301-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | I modified the way errors are returned, and this particular test is now getting a different access denied response. The new text happens to be what I intended to have here, so update the test. Change-Id: I53f8410ca0a52755d80473cd5cbcdb4d8502febf
* | | Merge "RepositoryState: add method canAmend"Christian Halstrick2011-11-301-0/+18
|\ \ \
| * | | RepositoryState: add method canAmendJens Baumgart2011-11-281-0/+18
| | |/ | |/| | | | | | | | | | | | | | | | | | | The method canAmend was added to RepositoryState. It returns true if amending the HEAD commit is allowed in the current repository state. Change-Id: Idd0c4eea83a23c41340789b7b877959b457d951e Signed-off-by: Jens Baumgart <jens.baumgart@sap.com>
* | | Merge "Always checkout master when it matches the advertised HEAD"Shawn Pearce2011-11-282-0/+22
|\ \ \
| * | | Always checkout master when it matches the advertised HEADKevin Sawicki2011-11-282-0/+22
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This parallels the CGit behavior of always using refs/heads/master when it matches the remote advertised HEAD commit. Change-Id: I5a5cd1516b58d116e334056aba1ef7990697ec30
* | | | Merge "Update maven plugin versions"Shawn Pearce2011-11-282-12/+12
|\ \ \ \
| * | | | Update maven plugin versionsMatthias Sohn2011-11-292-12/+12
| |/ / / | | | | | | | | | | | | | | | | Change-Id: I7400e08a1059f57c85a53aebe2719f81c00f58e8 Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
* | | | Merge "Implement Serializable interface in ReflogEntry"Shawn Pearce2011-11-281-1/+5
|\ \ \ \
| * | | | Implement Serializable interface in ReflogEntryKevin Sawicki2011-11-281-1/+5
| |/ / / | | | | | | | | | | | | Change-Id: Idf798dd3981bef3dc9e17c13c12809f89089e96f
* / / / Remove calls to printStackTrace in catch blocksKevin Sawicki2011-11-283-6/+1
|/ / / | | | | | | | | | Change-Id: I7a4179f10a4841e80b6546e1e7662cab71eac5e9
* | | Merge "Reset SSH connection and credentials on "Auth fail""Shawn Pearce2011-11-262-20/+55
|\ \ \
| * | | Reset SSH connection and credentials on "Auth fail"Matthias Sohn2011-11-272-20/+55
| |/ / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When SSH user/password authentication failed this may have been caused by changed credentials on the server side. When the SSH credentials of a user change the SSH connection needs to be re-established and credentials which may have been stored by the credentials provider need to be reset in order to enable prompting for the new credentials. Bug: 356233 Change-Id: I7d64c5f39b68a9687c858bb68a961616eabbc751 Signed-off-by: Matthias Sohn <matthias.sohn@sap.com>
* / / Don't iterate over advertised refs when HEAD is nullKevin Sawicki2011-11-261-3/+3
|/ / | | | | | | | | | | | | Moves the check from inside the loop to outside the loop and returns immediately if the HEAD advertisded ref is null Change-Id: I539da6cafb4f73610b8e00259e32bd4d57f4f4cc
* | Merge "tools/release: Handle v1.0.0.201106090707-r-NN-gdeadbeef"Matthias Sohn2011-11-241-1/+1
|\ \
| * | tools/release: Handle v1.0.0.201106090707-r-NN-gdeadbeefShawn O. Pearce2011-06-241-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | The 1.0.0 release tags have a new suffix. Account for this. Change-Id: Ic6f260b6a5ba353af3b312b722f576155208eaa0 Signed-off-by: Shawn O. Pearce <spearce@spearce.org>
* | | Merge "Adapt merge message to C Git for remote-tracking branch"Shawn Pearce2011-11-232-4/+4
|\ \ \
| * | | Adapt merge message to C Git for remote-tracking branchRobin Stocker2011-11-232-4/+4
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Commit 13931236b9ee2895a98ffdbdacbd0f895956d8a8 in C Git (2011-11-02) changed the message format: -Merge remote branch 'origin/foo' +Merge remote-tracking branch 'origin/foo' This change does the same in EGit to be consistent. Change-Id: I7d9c5afa95771dbfe6079b5f89a10b248fee0172 Signed-off-by: Robin Stocker <robin@nibor.org>