about summary refs log tree commit diff
path: root/.gitmodules (unfollow)
Commit message (Expand)Author
2009-01-11ui-log: show name-decorations in log outputLars Hjemli
2009-01-10Makefile: avoid libcurl when building gitLars Hjemli
2009-01-10ui-refs: avoid SEGFAULT on lightweight tagsRobin Redeker
2009-01-06ui-tree.c: do not add blank line when displaying blobsLars Hjemli
2008-12-30ui-patch: whitespace changes in the patch generation codeTomas Carnecky
2008-12-28tests/t0107-snapshot.sh: make testscript match updated snapshot behaviorLars Hjemli
2008-12-28return 404 if snapshot is not foundNatanael Copa
2008-12-26ui-snapshot.c: change mime-type for tar.gz and tar.bz2Lars Hjemli
2008-12-26Use GIT-1.6.1Lars Hjemli
2008-12-26shared.c: future-proof usage of git diff-structuresLars Hjemli
2008-12-26Untie logic for SCRIPT_NAME and PATH_INFOLars Hjemli
2008-12-06tests/setup.sh: allow testsuite to fail properly with POSIX standard shellsLars Hjemli
2008-12-06tests/t0010-validate-html.sh: skip tests if 'tidy' is not availableLars Hjemli
2008-12-05parsing.c: enable builds with NO_ICONV definedLars Hjemli
2008-12-05ui-repolist: avoid build warning for strcasestr(3)Lars Hjemli
2008-12-05Makefile: allow cgit.conf to override platform-specific tweaksLars Hjemli
2008-12-03Extra cygwin-specific changesRamsay Jones
2008-12-03ui-log: try to disambiguate ref namesLars Hjemli
2008-12-01ui-shared: shorten the sha1 printed by cgit_object_linkLars Hjemli
2008-12-01ui-refs.c: show download links for all tags referring to commit objectsLars Hjemli
2008-12-01ui-shared: exploit snapshot dwimmery in cgit_print_snapshot_linksLars Hjemli
2008-12-01ui-snapshot: improve extraction of revision from snapshot nameLars Hjemli
2008-11-30Set prefix in snapshots when using dwimmeryNatanael Copa
2008-11-30ui-log: use css to make full-log prettierLars Hjemli
2008-11-29ui-log: (ab)use extra columns for commit message when showmsg=1Lars Hjemli
2008-11-29ui-log: add support for showing the full commit messageLars Hjemli
2008-11-29ui-repolist: implement lazy caching of repo->mtimeLars Hjemli
2008-11-29ui-repolist: sort null values lastLars Hjemli
2008-11-29ui-repolist: add support for sorting any columnLars Hjemli
2008-11-29ui-repolist: extract get_repo_modtime() from print_modtime()Lars Hjemli
2008-11-29Add support for sorting by Age in the repolistBenjamin Close
2008-11-14ui-repolist: handle empty agefilesLars Hjemli
2008-11-06Use mode 0644 for non-executable filesKarl Chen
2008-11-06Fix tests to work on Ubuntu (dash)Ramsay Jones
2008-11-06Fix some warnings to allow -WerrorRamsay Jones
2008-11-06Use GIT-1.6.0.3Lars Hjemli
2008-10-11CGIT 0.8.1Lars Hjemli
2008-10-11Makefile: enable compilation on uclibcLars Hjemli
2008-10-11ui-snapshot: add dwimmeryLars Hjemli
2008-10-11Add cgit_query.nohead flagLars Hjemli
2008-10-06ui-shared: specify correct css class for summary tabLars Hjemli
2008-10-05CGIT 0.8Lars Hjemli
2008-10-05ui-summary: use html_url_path()Lars Hjemli
2008-10-05ui-refs: use cgit_tag_link()Lars Hjemli
2008-10-05ui-shared: add cgit_tag_link()Lars Hjemli
2008-10-05ui-shared: generate proper links in cgit_object_link()Lars Hjemli
2008-10-05ui-shared: use html_url_path() to get properly escaped url in form actionLars Hjemli
2008-10-05Use GIT-1.6.0.2Lars Hjemli
2008-10-05ui-shared: reword the standard page footerLars Hjemli
2008-10-05ui-shared: do not print repo name on the "summary" tabLars Hjemli
generated by cgit-pink 1.4.1 (git 2.36.1) at 2026-07-31 21:24:40 +0000
.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-19redir: Fix typo in noclobber codeHerbert Xu The noclobber code has a typo in it that causes it to fail. This patch fixes it. Reported-by: Denys Vlasenko <vda.linux@googlemail.com> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-19expand: Fix glibc glob(3) supportHerbert Xu It's been a while since we disabled glob(3) support by default. It appears to be working now, however, we have to change our code to detect the no-match case correctly. In particular, we need to test for GLOB_NOMAGIC | GLOB_NOCHECK instead of GLOB_MAGCHAR. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02expand: Fix buffer overflow in expandmetaHerbert Xu The native version of expandmeta allocates a buffer that may be overrun for two reasons. First of all the size is 1 byte too small but this is normally hidden because the minimum size is rounded up to 2048 bytes. Secondly, if the directory level is deep enough, any buffer can be overrun. This patch fixes both problems by calling realloc when necessary. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02builtin: Move echo space/nl handling into print_escape_strHerbert Xu Currently echocmd uses print_escape_str to do everything apart from printing the spaces/newlines separating its arguments. This patch moves the actual printing into print_escape_str as well using the format parameter. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02builtin: Fix echo performance regressionHerbert Xu The commit d6c0e1e2ffbf7913ab69d51cc794d48d41c8fcb1 ("[BUILTIN] Handle embedded NULs correctly in printf") caused a performance regression in the echo built-in because every echo call now goes through the printf %b slow path where the string is always printed twice to ensure the space padding is correct in the presence of NUL characters. In fact this regression applies to printf %b as well. This is easily fixed by making printf %b take the fast path when no precision/field width modifiers are present. This patch also changes the second strchurnul call to strspn which generates slightly better code. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02expand: Fix ghost fields with unquoted $@/$*Herbert Xu Harald van Dijk <harald@gigawatt.nl> wrote: > On 22/03/2018 22:38, Martijn Dekker wrote: >> Op 22-03-18 om 20:28 schreef Harald van Dijk: >>> On 22/03/2018 03:40, Martijn Dekker wrote: >>>> This patch fixes the bug that, given no positional parameters, unquoted >>>> $@ and $* incorrectly generate one empty field (they should generate no >>>> fields). Apparently that was a side effect of the above. >>> >>> This seems weird though. If you want to remove the recording of empty >>> regions because they are pointless, then how does removing them fix a >>> bug? Doesn't this show that empty regions do have an effect? Perhaps >>> they're not supposed to have any effect, perhaps it's a specific >>> combination of empty regions and something else that triggers some bug, >>> and perhaps that combination can no longer occur with your patch. >> >> The latter is my guess, but I haven't had time to investigate it. > > Looking into it again: > > When IFS is set to an empty string, sepc is set to '\0' in varvalue(). > This then causes *quotedp to be set to true, meaning evalvar()'s quoted > variable is turned on. quoted is then passed to recordregion() as the > nulonly parameter. > > ifsp->nulonly has a bigger effect than merely selecting whether to use > $IFS or whether to only split on null bytes: in ifsbreakup(), nulonly > also causes string termination to be suppressed. That's correct: that > special treatment is required to preserve empty fields in "$@" > expansion. But it should *only* be used when $@ is quoted: ifsbreakup() > takes nulonly from the last IFS region, even if it's empty, so having an > additional zero-length region with nulonly enabled causes confusion. > > Passing quoted by value to varvalue() and not attempting to modify it > should therefore, and in my quick testing does, also work to fix the > original $@ bug. You're right. The proper fix to this is to ensure that nulonly is not set in varvalue for $*. It should only be set for $@ when it's inside double quotes. In fact there is another bug while we're playing with $@/$*. When IFS is set to a non-whitespace character such as :, $* outside quotes won't remove empty fields as it should. This patch fixes both problems. Reported-by: Martijn Dekker <martijn@inlv.org> Suggested-by: Harald van Dijk <harald@gigawatt.nl> Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02parser: Allow newlines within parameter substitutionHerbert Xu On Fri, Mar 16, 2018 at 11:27:22AM +0800, Herbert Xu wrote: > On Thu, Mar 15, 2018 at 10:49:15PM +0100, Harald van Dijk wrote: > > > > Okay, it can be trivially modified to something that does work in other > > shells (even if it were actually executed), but gets rejected at parse time > > by dash: > > > > if false; then > > : ${$+ > > } > > fi > > That's just a bug in dash's parser with ${} in general, because > it bombs out without the if clause too: > > : ${$+ > } This patch fixes the parsing of newlines with parameter substitution. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-04-02expand: Fix bugs with words connected to the right of $@Herbert Xu On Sun, Mar 04, 2018 at 12:44:59PM +0100, Harald van Dijk wrote: > > command: set -- a ""; space=" "; printf "<%s>" "$@"$space > bash: <a><> > dash 0.5.8: <a>< > > dash 0.5.9.1: <a>< > > dash patched: <a><> This is actually composed of two bugs. First of all our tracking of quotemark is wrong so anything after "$@" becomes quoted. Once we fix that then the problem is that the first space character after "$@" is not recognised as an IFS. This patch fixes both. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-03-25Revert "[BUILTIN] Remove unnecessary restoration of format string in printf"Herbert Xu This reverts commit 7bb413255368e94395237d789f522891093c5774. The commit breaks printf with more than argument. Signed-off-by: Herbert Xu <herbert@gondor.apana.org.au> 2018-03-22parser: Fix backquote support in here-document EOF markHerbert Xu