summaryrefslogtreecommitdiff
Commit message (Collapse)AuthorAgeFilesLines
* Round up pool alloc sizes for alignmentrb/round-up-pool-allocationsRussell Belfer2014-06-302-6/+6
| | | | | | | | | | | | | To make sure that items returned from pool allocations are aligned on nice boundaries, this rounds up all pool allocation sizes to a multiple of 8. This adds a small amount of overhead to each item. The rounding up could be made optional with an extra parameter to the pool initialization that turned on rounding only for pools where item alignment actually matters, but I think for the extra code and complexity that would be involved, that it makes sense just to burn a little bit of extra memory and enable this all the time.
* Merge pull request #2440 from phkelley/transportsVicent Marti2014-06-3010-68/+177
|\ | | | | Improvements to git_transport extensibility
| * Improvements to git_transport extensibilityPhilip Kelley2014-06-2610-68/+177
| | | | | | | | | | git_remote_set_transport now takes a transport factory rather than a transport git_clone_options now allows the caller to specify a remote creation callback
* | Merge pull request #2447 from phkelley/pkt_assertVicent Marti2014-06-272-1/+8
|\ \ | | | | | | Fix assert when receiving uncommon sideband packet
| * | Fix assert when receiving uncommon sideband packetPhilip Kelley2014-06-272-1/+8
|/ /
* | Merge pull request #2445 from ethomson/checkout_conflict_testVicent Marti2014-06-261-1/+1
|\ \ | | | | | | checkout::conflict tests: only test owner mode
| * | checkout::conflict tests: only test owner modeEdward Thomson2014-06-261-1/+1
|/ / | | | | | | | | | | | | The checkout::conflict type conflict tests were failing because they were overly assertive about the resultant mode, testing group & other bits, which failed miserably for people who had a umask less restrictive than 022. Only test the resultant owner bits.
* | Merge pull request #2444 from libgit2/phkelley/flexarrayPhilip Kelley2014-06-261-1/+1
|\ \ | |/ |/| Fixes #2443 Zero size arrays are an extension
| * Fixes #2443 Zero size arrays are an extensionphkelley/flexarrayPhilip Kelley2014-06-261-1/+1
|/
* Merge pull request #2442 from libgit2/cmn/leaksVicent Marti2014-06-254-17/+20
|\ | | | | Fix a couple of leaks
| * pack: clean up error returnsCarlos Martín Nieto2014-06-252-13/+11
| | | | | | | | | | | | | | | | | | Set a message when we fail to lock. Also make the put function void, since it's called from free, which cannot report errors. The only errors we can experience here are internal state corruption, so we assert that we are trying to put a pack which we have previously got.
| * tree: free in error conditionsCarlos Martín Nieto2014-06-251-0/+2
| | | | | | | | As reported by coverity, we would leak some memory in error conditions.
| * pack: free the new pack struct if we fail to insertCarlos Martín Nieto2014-06-252-4/+7
|/ | | | | | | | If we fail to insert the packfile in the map, make sure to free it. This makes the free function only attempt to remove its mwindows from the global list if we have opened the packfile to avoid accessing the list unlocked.
* travis: build the master branchCarlos Martín Nieto2014-06-251-1/+2
| | | | | We need to tell Travis to build the master branch or it won't build it or the pull requests.
* Merge pull request #2435 from ethomson/text_attrVicent Marti2014-06-253-3/+12
|\ | | | | Update text=auto / core.autocrlf=false behavior
| * Update text=auto / core.autocrlf=false behaviorEdward Thomson2014-06-243-3/+12
| | | | | | | | | | | | | | Git for Windows 1.9.4 changed the behavior when the text=auto attribute is specified and core.autocrlf=false. Previous observed behavior would *not* filter files when going into the working directory, the new behavior *does* filter. Update our behavior to match.
* | Merge pull request #2439 from ethomson/revert_exportVicent Marti2014-06-251-1/+1
|\ \ | |/ |/| Export git_revert_commit
| * Export git_revert_commitEdward Thomson2014-06-251-1/+1
|/
* Add CHANGELOG entry for mixed-EOL fixCarlos Martín Nieto2014-06-241-0/+2
|
* Merge remote-tracking branch 'upstream/cmn/mixed-eol-passthrough'Carlos Martín Nieto2014-06-244-33/+22
|\
| * crlf: pass-through mixed EOL buffers from LF->CRLFcmn/mixed-eol-passthroughCarlos Martín Nieto2014-06-234-33/+22
| | | | | | | | | | | | | | | | | | | | When checking out files, we're performing conversion into the user's native line endings, but we only want to do it for files which have consistent line endings. Refuse to perform the conversion for mixed-EOL files. The CRLF->LF filter is left as-is, as that conversion is considered to be normalization by git and should force a conversion of the line endings.
* | Add CHANGELOG entry for treebuilder mapCarlos Martín Nieto2014-06-241-0/+2
| |
* | Merge remote-tracking branch 'upstream/cmn/treebuilder-perf'Carlos Martín Nieto2014-06-244-62/+72
|\ \
| * | treentry: no need for manual size book-keepingcmn/treebuilder-perfCarlos Martín Nieto2014-06-102-10/+6
| | | | | | | | | | | | We can simply ask the hasmap.
| * | treebuilder: don't keep removed entries aroundCarlos Martín Nieto2014-06-103-38/+22
| | | | | | | | | | | | | | | | | | If the user wants to keep a copy for themselves, they should make a copy. It adds unnecessary complexity to make sure the returned entries are valid until the builder is cleared.
| * | treebuilder: use a map instead of vector to store the entriesCarlos Martín Nieto2014-06-104-54/+84
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Finding a filename in a vector means we need to resort it every time we want to read from it, which includes every time we want to write to it as well, as we want to find duplicate keys. A hash-map fits what we want to do much more accurately, as we do not care about sorting, but just the particular filename. We still keep removed entries around, as the interface let you assume they were going to be around until the treebuilder is cleared or freed, but in this case that involves an append to a vector in the filter case, which can now fail. The only time we care about sorting is when we write out the tree, so let's make that the only time we do any sorting.
* | | Add CHANGELOG entry for shared packsCarlos Martín Nieto2014-06-241-0/+2
| | |
* | | Merge branch 'cmn/global-mwf'Carlos Martín Nieto2014-06-247-14/+202
|\ \ \
| * | | Share packs across repository instancescmn/global-mwfCarlos Martín Nieto2014-06-237-14/+202
| | |/ | |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | Opening the same repository multiple times will currently open the same file multiple times, as well as map the same region of the file multiple times. This is not necessary, as the packfile data is immutable. Instead of opening and closing packfiles directly, introduce an indirection and allocate packfiles globally. This does mean locking on each packfile open, but we already use this lock for the global mwindow list so it doesn't introduce a new contention point.
* | | Merge pull request #2434 from libgit2/rb/contributing-updatesVicent Marti2014-06-242-33/+75
|\ \ \ | | | | | | | | Update CONTRIBUTING and PROJECTS
| * | | missing parenrb/contributing-updatesRussell Belfer2014-06-201-1/+1
| | | |
| * | | Update CONTRIBUTING and PROJECTSRussell Belfer2014-06-202-33/+75
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This updates CONTRIBUTING to reflect the changes to use the master branch and make explicit recommendations about updating CHANGELOG.md and providing test coverage. Also, this includes some old updates to PROJECTS.md that I wrote to expand the list of projects, pulling in things from old feature requests.
* | | | Add a CHANGELOG entry for the filebuf changeCarlos Martín Nieto2014-06-231-0/+5
| | | |
* | | | Merge remote-tracking branch 'upstream/cmn/filebuf-atomic-unlock'Carlos Martín Nieto2014-06-232-4/+28
|\ \ \ \ | |/ / / |/| | |
| * | | filebuf: make unlocking atomiccmn/filebuf-atomic-unlockCarlos Martín Nieto2014-06-042-4/+28
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When renaming a lock file to its final location, we need to make sure that it is replaced atomically. We currently have a workaround for Windows by removing the target file. This means that the target file, which may be a ref or a packfile, may cease to exist for a short wile, which shold be avoided. Implement the workaround only in Windows, by making sure that the file we want to replace is writable.
* | | | libgit2 v0.21.0v0.21.0Vicent Marti2014-06-20784-10716/+31834
|\ \ \ \ | | |/ / | |/| |
| * | | Merge pull request #2425 from libgit2/cmn/clone-auth-changev0.21.0-rc2Vicent Marti2014-06-161-9/+3
| |\ \ \ | | | | | | | | | | Don't use read-write accounts for the tests
| | * | | test: remove assembla clone testcmn/clone-auth-changeCarlos Martín Nieto2014-06-131-6/+0
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The assembla failure we were seeing referred to a private repository, which is not what is there at the moment. This reverts 1fd21b0342f
| | * | | test: use read-only accountCarlos Martín Nieto2014-06-131-3/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Don't write in plaintext the password of an account which has full control over the repository. Instead use an account with read-only access.
| * | | | Merge pull request #2421 from libgit2/cmn/init-ssl-onceVicent Marti2014-06-144-16/+81
| |\ \ \ \ | | |/ / / | |/| | | netops: init OpenSSL once under lock
| | * | | ssl: init everything all the timecmn/init-ssl-onceCarlos Martín Nieto2014-06-123-87/+51
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Bring together all of the OpenSSL initialization to git_threads_init() so it's together and doesn't need locks. Moving it here also gives us libssh2 thread safety (when built against openssl).
| | * | | ssl: init also without threadsCarlos Martín Nieto2014-06-121-9/+10
| | | | |
| | * | | ssl: cargo-cult thread safetyCarlos Martín Nieto2014-06-124-23/+46
| | | | | | | | | | | | | | | | | | | | OpenSSL's tests init everything in the main thread, so let's do that.
| | * | | ssl: use lockingCarlos Martín Nieto2014-06-111-0/+42
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When using in a multithreaded context, OpenSSL needs to lock, and leaves it up to application to provide said locks. We were not doing this, and it's just luck that's kept us from crashing up to now.
| | * | | netops: init OpenSSL once under lockCarlos Martín Nieto2014-06-113-2/+37
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The OpenSSL init functions are not reentrant, which means that running multiple fetches in parallel can cause us to crash. Use a mutex to init OpenSSL, and since we're adding this extra checks, init it only once.
| * | | | http: fix typo in credentials logicCarlos Martín Nieto2014-06-131-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | We want to check whether the credentials callback is NULL, not whether the payload is.
| * | | | remote: update documentationCarlos Martín Nieto2014-06-111-18/+60
| | | | | | | | | | | | | | | | | | | | | | | | | Add docs for git_clone_local_t and move the docs for the git_clone_options to each field.
| * | | | Merge pull request #2417 from libgit2/cmn/revwalk-array-fixVicent Marti2014-06-111-2/+2
| |\ \ \ \ | | |_|_|/ | |/| | | revwalk: more sensible array handling
| | * | | revwalk: more sensible array handlingcmn/revwalk-array-fixCarlos Martín Nieto2014-06-111-2/+2
| |/ / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Instead of using a sentinel empty value to detect the last commit, let's check for when we get a NULL from popping the stack, which lets us know when we're done. The current code causes us to read uninitialized data, although only on RHEL/CentOS 6 in release mode. This is a readability win overall.
| * | | Merge pull request #2416 from libgit2/cmn/treebuilder-insert-sortedVicent Marti2014-06-101-2/+2
| |\ \ \ | | | | | | | | | | treebuilder: insert sorted