aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeLines
* Avoid trailing slash in virtual-rootGravatar Lars Hjemli2011-03-05-2/+3
| | | | | | | | | | When setting virtual-root from cgitrc, care is taken to avoid trailing slashes. But when no virtual-root setting is specified, SCRIPT_FILE from the web server is used without similar checks. This patch fixes the inconsistency, which could lead to double-slashes in generated links. Noticed-by: Wouter Van Hemel <wouter@duodecim.org> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* do not infloop on a query ending in %XY, for invalid hex X or YGravatar Jim Meyering2011-03-05-1/+1
| | | | | | | | | | | When a query ends in say %gg, (or any invalid hex) e.g., http://git.gnome.org/browse/gdlmm/commit/?id=%gg convert_query_hexchar calls memmove(txt, txt+3, 0), and then returns txt-1, so the loop in http_parse_querystring never terminates. The solution is to make the memmove also copy the trailing NUL. * html.c (convert_query_hexchar): Fix off-by-one error. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-shared: silence warningGravatar Bernhard Reutner-Fischer2011-02-19-1/+1
| | | | | | | | warning: format ‘%ld’ expects type ‘long int’, but argument 2 has type ‘size_t’ Signed-off-by: Bernhard Reutner-Fischer <rep.dot.nop@gmail.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Makefile: Make `make get-git` work under OpenBSD.Gravatar Lukas Fleischer2011-02-19-1/+1
| | | | | | | | OpenBSD tar(1) defaults to read from "/dev/rst0" when not specifying an filename and thus fails to extract the Git sourcecode when not passing stdin as input file descriptor explicitly. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Use absolute path for scanned repo readmeGravatar Dean Scarff2010-11-08-1/+1
| | | | | | | | repo.readme is transformed to an absolute path when read from the config, so add_repo needs to add "README.html" with the repository path too. Signed-off-by: Dean Scarff <dos@scarff.id.au> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* CGIT 0.8.3.4Gravatar Lars Hjemli2010-09-27-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Use GIT-1.7.3Gravatar Lars Hjemli2010-09-27-8/+8
| | | | | | | This fixes http://web.nvd.nist.gov/view/vuln/detail?vulnId=CVE-2010-2542. Noticed-by: Silvio Cesare <silvio.cesare@gmail.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* RSS items should always use UTC timesGravatar Aaron Griffin2010-09-20-2/+2
| | | | | | | | The format uses a 'Z' suffix, which indicates no TZ offset. Thus we should not respect the local-time config setting here. Signed-off-by: Aaron Griffin <agriffin@datalogics.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-snapshot: actually compress zip archivesGravatar Lars Hjemli2010-09-01-0/+1
| | | | | | | Since cgit linked with git-1.6.0 all zip archives has been uncompressed. This patch fixes the issue by specifying Z_DEFAULT_COMPRESSION. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* html: fix strcpy bug in convert_query_hexcharGravatar Mark Lodato2010-08-30-4/+5
| | | | | | | | The source and destination strings in strcpy() may not overlap. Instead, use memmove(), which allows overlap. This fixes test t0104, where 'url=foo%2bbar/tree' was being parsed improperly. Signed-off-by: Mark Lodato <lodatom@gmail.com>
* t0108-patch: add 'tests_done' to endGravatar Mark Lodato2010-08-30-0/+2
| | | | Signed-off-by: Mark Lodato <lodatom@gmail.com>
* CGIT 0.8.3.3Gravatar Lars Hjemli2010-08-04-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-refs.c: avoid segfault on unparsed ref objectsGravatar Lars Hjemli2010-08-04-12/+14
| | | | | | | | When a ref refers to something other then a commit or tag object, cgit could segfault when trying to display the tag info. Noticed-by: Eugene Sajine <euguess@gmail.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Reencode author and committerGravatar Rémi Lagacé2010-07-14-0/+4
| | | | | | | When a commit has a specific encoding, this encoding also applies to the author and committer name and email. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* CGIT 0.8.3.2Gravatar Lars Hjemli2010-06-19-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* shared.c: avoid memory leak during diffGravatar Lars Hjemli2010-04-08-0/+8
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Clear the whole contextGravatar Daniel Milde2010-02-27-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-shared.c: fix segfault when repo is emptyGravatar Florian Pritz2010-02-27-2/+2
| | | | | | | | | PATH_INFO="/$REPONAME/commit/?id=1" QUERY_STRING="id=1" ./cgit.cgi triggers segfault when the repository is empty and therefore ctx.qry.head is unset Signed-off-by: Florian Pritz <bluewind@xssn.at> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* CGIT 0.8.3.1Gravatar Lars Hjemli2009-12-12-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Fix segfault on ppc when browsing treeGravatar Martins Polakovs2009-12-12-3/+3
|
* Don't crash when a repo-specific readme file is usedGravatar Sami Kyöstilä2009-12-09-2/+2
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* shared.c: return original errnoGravatar Lars Hjemli2009-11-08-3/+5
| | | | | Noticed-by: Andreas Schwab <schwab@linux-m68k.org> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Close fd on error in readfile()Gravatar Rys Sommefeldt2009-11-07-2/+7
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Nov is the correct abbreviationGravatar Danijel Tašov2009-11-07-1/+1
|
* Fix repolist search links with virtual rootGravatar Geoff Johnstone2009-09-20-1/+2
| | | | | Signed-off-by: Geoff Johnstone <geoff.johnstone@googlemail.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* cgitrc.5.txt: Change repo.group to section in example config.Gravatar Loui Chang2009-09-15-4/+4
| | | | | Signed-off-by: Loui Chang <louipc.ist@gmail.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* cgitrc.5.txt: Add mansource and manmanual.Gravatar Loui Chang2009-09-15-0/+3
| | | | | | | This prevents FIXMEs from appearing. Signed-off-by: Loui Chang <louipc.ist@gmail.com> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* CGIT 0.8.3Gravatar Lars Hjemli2009-09-14-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Merge branch 'stable'Gravatar Lars Hjemli2009-09-14-1/+1
|\
| * CGIT 0.8.2.2Gravatar Lars Hjemli2009-09-14-1/+1
| | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | Merge branch 'lh/repo-scan'Gravatar Lars Hjemli2009-09-14-80/+329
|\ \
| * | cgit.c: respect repo-local 'snapshots' option for --scan-pathGravatar Lars Hjemli2009-08-24-0/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The repo-specific 'snapshots' option is bitwise AND'ed with the global 'snapshots' option during parsing, and since the global cgitrc hasn't been parsed when --scan-path is processed the global 'snapshots' will always be 0 (i.e. no repo-specific 'snapshots' setting will have any effect). This patch fixes the issue by setting the global 'snapshots' mask to 0xFF (hence relying on later parsing of the generated cgitrc repolist to do the right thing). Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.c: only print first line of repo.desc in print_repo()Gravatar Lars Hjemli2009-08-24-2/+14
| | | | | | | | | | | | | | | | | | | | | | | | Since repo.desc might have been populated by reading the 'description' file in GIT_DIR, it may contain newlines. And by printing the literal value, we may then generate an invalid cgitrc include-file. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Add and use cgit_find_stats_periodname() in print_repo()Gravatar Lars Hjemli2009-08-24-0/+12
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.c: generate repo.snapshots in print_repo()Gravatar Lars Hjemli2009-08-24-0/+25
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.c: add missing options to print_repo()Gravatar Lars Hjemli2009-08-24-0/+18
| | | | | | | | | | | | | | | | | | | | | | | | Note: print_repo() still ignores repo.max-stats and repo.snapshots, which both requires additional work since these settings are represented internally as an enum and a bitmap. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | shared.c: initialize cgit_repo structs properlyGravatar Lars Hjemli2009-08-24-0/+1
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Add config option 'enable-filter-overrides'Gravatar Lars Hjemli2009-08-24-11/+26
| | | | | | | | | | | | | | | | | | | | | This option must be enabled if repo-specific cgitrc files should be allowed to override any of the 'filter' options. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgitrc.5.txt: fix markup errorsGravatar Lars Hjemli2009-08-24-9/+10
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | ui-repolist: handle empty sections similar to NULL sectionsGravatar Lars Hjemli2009-08-24-7/+12
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Add support for repo-local cgitrc fileGravatar Lars Hjemli2009-08-24-13/+39
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When recursively scanning a directory tree looking for git repositories, cgit will now parse cgitrc files found within such repositories. The repo-specific config files can include any repo-specific options except 'repo.url' and 'repo.path'. Also, in such config files the 'repo.' prefix can not be used, i.e. the valid options then becomes: * name * clone-url * desc * ower * defbranch * snapshots * enable-log-filecount * enable-log-linecount * max-stats * module-link * section * about-filter * commit-filter * source-filter * readme Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.c: refactor repo_config() from config_cb()Gravatar Lars Hjemli2009-08-24-41/+48
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new function repo_config() is used to handle all 'simple' repo options, for the following reasons: * code readability * parser performance * upcoming support for repo-local cgitrc files during scanning Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | ui-repolist.c: sort by section name, repo name as defaultGravatar Lars Hjemli2009-08-24-0/+15
| | | | | | | | | | | | | | | | | | | | | | | | When no sorting is requested by the client, cgit will now sort by section name followed by repo name. This allows repos to be registered/ discovered independently of their display order. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Add config option 'repo.section'Gravatar Lars Hjemli2009-08-24-0/+5
| | | | | | | | | | | | | | | | | | | | | This option can be used to specify the section name for the current repository. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Introduce 'section' as canonical spelling for 'repo.group'Gravatar Lars Hjemli2009-08-24-17/+22
| | | | | | | | | | | | | | | | | | | | | | | | The 'repo.' prefix should be reserved for repo-specific options, but the option 'repo.group' must still be honored to stay backwards compatible. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Add support for --scan-path command line optionGravatar Lars Hjemli2009-08-24-1/+2
| | | | | | | | | | | | | | | | | | | | | This is an alias for --scan-tree (which might be deprecated in the future). Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Rename 'repo.scan' to 'scan-path'Gravatar Lars Hjemli2009-08-24-10/+10
| | | | | | | | | | | | | | | | | | | | | The 'repo.' prefix will soon be reserved for repo-specific config options. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgitrc.5.txt: document repo.scan and cache-scanrc-ttlGravatar Lars Hjemli2009-08-24-0/+9
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.c: add support for caching autodetected repositoriesGravatar Lars Hjemli2009-08-24-1/+75
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.c: make print_repolist() and print_repo() reusable for cachingGravatar Lars Hjemli2009-08-24-12/+12
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>