Commits
- Commit:
7e91f3d2e1322160a4db79cd753888592c6fbf18
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
read_gitconfig: fix GOT_IGNORE_GITCONFIG case
set gitconfig_remotes to NULL to avoid a double-free, and reset the
global author name/email fields too.
ok stsp@
- Commit:
5daf5c53287d227ebc29ad1b0d934b4731ad5990
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
add a way to ignore author and remote repository info found in Git config files
ok op@
- Commit:
7c60996cbac298efd8023b1b599d13371ae61963
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
comment tweak
- Commit:
0b1c1d9c3d925795955ba06a2ea73228e1c1d112
- From:
- Lucas <lucas@sexy.is>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
got_repo_pack_fds_close: don't close(0)
got_repo_pack_fds_close will try to close all of the GOT_PACK_NUM_TEMPFILES fds,
even the ones that didn't manage to get a value from got_opentempfd(), which
would result in a close(0) if we do not initialize to -1 here.
- Commit:
cfc8922bd7eff82e98dcc2d45945769a1a76806c
- From:
- Florian Obser <florian@narrans.de>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
Make sure got_repo_pack_fds_close() frees a malloc'ed pointer.
Found by llvm's scan-build (bad free).
OK stsp
- Commit:
ebb42948df126b79ef0cff4ffb379c748f166ec0
- From:
- Florian Obser <florian@narrans.de>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
Don't chug along if repo format version is unsupported.
Found by llvm's scan-build (dead store).
OK stsp
- Commit:
65ad15fa440d89e976ce1bd3bd02d5440e86d5d3
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: lib/repository.c: fixup merge
It seems as though a merge from some time ago had left some stale
changes in place. This brings the two files between upstream and
portable back in-line.
Noticed by stsp@
- Commit:
8b925c6ccd3968917253ea191906a1710c40ed2d
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: add back sys/queue.h
Now that the handling of including sys/queue.h is better, there's no
need to remove those lines from the source. Copy the location of those
original sys/queue.h lines from upstream at the same line number, so as
to avoid any conflicts in the future.
- Commit:
ec2b23c5d2570888a1d83ce2e8eee5a90eeff695
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
implement support for commit coloring in got-read-pack for speed
ok op, tracey
- Commit:
0dcca008649b39a379b76232af4afac403b9c483
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
remove pointless assignment at the end of got_repo_pack_fds_close()
- Commit:
7648b8ae7289c7590d1cfac906eaebaacb00e1c4
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
don't look for close() errors if opentemp() failed in got_repo_pack_fds_open()
- Commit:
7cd528337bab10d64ef73bb1a6e40c6203539e4d
- From:
- Tracey Emery <tracey@traceyemery.net>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
move got_opentempfd out of got_repo_open. ok stsp@
thanks for all the help massaging this diff
- Commit:
9394843506b5859aab24498152a3adcf21a76b12
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
properly swap cached struct pack array elements in got_repo_cache_pack()
Avoids clobbering open files for delta base/accumulation, leaking file
descriptors, and triggering errors from close(2) during got_repo_close()
as we try to close the same file descriptor more than once.
- Commit:
cf032c44a2f1308d52ec68df3417e409f262ed45
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
ensure that all open basefd/accumfd get closed in got_repo_close()
found by tracey
- Commit:
bfb5ee0b7372ae375143b9dee5552b8bec26cf05
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
open temporary files needed for delta application in got_repo_open()
This prepares for callers of got_repo_open() that cannot afford to
open files in /tmp, such as gotwebd. In a follow-up change, we could
ask such callers to pass in the required amount of open temporary files.
One consequence is that got_repo_open() now requires the "cpath" pledge
promise. Add the "cpath" promise to affected callers and remove it once
the repository has been opened.
ok tracey
- Commit:
ec242592d329728975bf10a1196907167de7fed0
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
inline struct got_object_id in struct got_object_qid
Saves us from doing a malloc/free call for every item on the list.
ok op@
- Commit:
d224ad331179af78d8a4f232001f0a6595e36d85
- From:
- Christian Weisgerber <naddy@mips.inka.de>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
const-ify string table
- Commit:
58cbcd83c93f2636813f4f8064caefd2f838a24c
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
don't `goto done' if calloc fails
otherwise we call got_repo_close with repo being NULL and segfault
there.
ok stsp
- Commit:
de47d040c7458d8bddac77ba76a8a5e61aa8eb44
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
cache a list of known pack index files when the repository is opened
Avoids overhead due to readdir calls while searching a pack index.
ok op@
- Commit:
70e6418ec50331f61e1c705e3c73a659b496bcdd
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
make got_repo_match_object_id_prefix() reject overlong input strings
suggested by + ok naddy
- Commit:
9d70a0bf7236fdcfbe7186f0d90a5a15f5a10b18
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: make UUID includes generic
Move the detection of UUID header files to be included portably. This
will have no functional effect until MacOS detection is put in place.
- Commit:
e2752401e9de43f1f8c1e3a1601ec073cccab548
- From:
- Omar Polo <op@omarpolo.com>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
factorize imsg_clear calls after imsg_flush failures
imsg_clear frees and closes resources allocated as part of enqueueing
imsgs so it's a no-op after reads.
discussed with and ok stsp@
- Commit:
9389bcf69b554cb6da81ece9452257fa11011dd2
- From:
- Thomas Adam <thomas@xteddy.org>
- Date:
portable: fix use of d->namlen
As done elsewhere, d->namlen is not a portable field from readdir,
therefore simualte this via the strlen of the directory name instead.
- Commit:
f9c2e8e5d5310b6b1cd22dcfc8f9f5d1d7a16d19
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
reuse existing deltas when creating pack files
tested by thomas, naddy, and myself
- Commit:
48b4f239031ad66225f64bd6745f4a57a35c3a0f
- From:
- Stefan Sperling <stsp@stsp.name>
- Via:
- Thomas Adam <thomas@xteddy.org>
- Date:
wrap overlong lines