aboutsummaryrefslogtreecommitdiffstats
Commit message (Collapse)AuthorAgeFilesLines
* ui-plain.c: only return the blob with the specified pathLars Hjemli2009-09-061-1/+2
| | | | | | | | | | When a path to a directory was specified for the 'plain' view, each blob in the directory used to be returned to the client. This patch fixes the issue by matching the path of each blob against the requested path. Noticed-by: Lars Stoltenow <penma@penma.de> Signed-off-by: Lars Hjemli <larsh@slackbox.hjemli.net>
* truncate buffer before reading empty filesSimon Arlott2009-08-182-0/+2
| | | | | | | | | | If readfile() reads an empty file, fgets() won't truncate the buffer and it'll still contain the contents of the previously read file. [lh: fixed similar issue in ui-repolist.c] Signed-off-by: Simon Arlott <simon@fire.lp0.eu> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-shared: don't print header <img/> if there isn't a logo definedMatthew Metnetsky2009-08-181-8/+11
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* cgit.c: do not segfault on unexpected query-string formatLars Hjemli2009-08-171-0/+3
| | | | | | | | The querystring_cb() function will be invoked with a NULL value when the querystring contains a name not followed by a '='. Such a value used to cause a segfault, which this patch fixes. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-tag.c: do not segfault when id is missing from query-stringLars Hjemli2009-08-171-1/+4
| | | | | | | | | | | | | | | The purpose of the tag page is to print info about a specific tag, but if no tag was specified on the query-string cgit used to segfault. With this patch, cgit will fallback to the value of the 'h' parameter instead (which is never NULL due to prepare_repo_cmd() in cgit.c). It will now also verify that the specified tagname is in fact a valid ref in the 'refs/tags/' namespace, i.e. specifying 'id=master' will trigger a 'Bad tag reference' error. Noticed-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* CGIT 0.8.2.1Lars Hjemli2009-03-151-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-snapshot: avoid segfault when no filename is specifiedLars Hjemli2009-03-151-6/+17
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* fix segfault when displaying empty blobsEric Wong2009-03-151-5/+8
| | | | | | | | When size is zero, subtracting one from it turns it into ULONG_MAX which causes an out-of-bounds access on buf. Signed-off-by: Eric Wong <normalperson@yhbt.net> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* ui-tree: escape ascii-text properly in hexdump viewLars Hjemli2009-02-121-4/+9
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* CGIT 0.8.2Lars Hjemli2009-02-011-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Use GIT-1.6.1.1Lars Hjemli2009-02-012-1/+1
| | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* Merge branch 'lh/binary'Lars Hjemli2009-02-017-30/+119
|\
| * shared.c: avoid SEGFAULT when checking for binary buffersLars Hjemli2009-02-011-2/+2
| | | | | | | | | | | | | | Before calling buffer_is_binary() we need to verify that the buffer is valid. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * ui-tree: show hexdump of binary blobsLars Hjemli2009-01-312-23/+67
| | | | | | | | | | | | | | This teaches ui-tree to detect binary blobs and display them similar to `hexdump -C` (only wider). Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * Handle binary files in diffsLars Hjemli2009-01-315-7/+52
| | | | | | | | | | | | | | This teaches all diff-related operations (i.e. ui-log, ui-diff and ui-patch) how to handle binary files. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | Merge branch 'lh/header'Lars Hjemli2009-02-014-0/+9
|\ \ | |/ |/|
| * Add support for a custom headerLars Hjemli2009-01-294-0/+9
| | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | Merge branch 'stable'Lars Hjemli2009-01-291-2/+4
|\ \ | |/ |/|
| * html.c: use correct escaping in html attributesLars Hjemli2009-01-291-2/+4
| | | | | | | | | | | | | | First, an apostrophe is not a quote. Second, we also need to escape quotes. And finally, quotes are encoded as '&quot;', not '&quote;'. Sighned-off-by: Lars Hjemli <hjemli@gmail.com>
* | Merge branch 'stable'Lars Hjemli2009-01-271-1/+1
|\|
| * CGIT 0.8.1.1Lars Hjemli2009-01-271-1/+1
| | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | Merge branch 'jw/commit-decorations'Lars Hjemli2009-01-274-1/+12
|\ \ | | | | | | | | | | | | | | | | | | Conflicts: cgit.css Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | tests/t0105-commit.sh: expect commit decorationsLars Hjemli2009-01-271-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | When a commit is referenced by a branch or tag the commit subject- line will be suffixed with the proper decorations, hence the test for this line needs to be updated. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | cgit.css: minor adjustment of commit decorationsLars Hjemli2009-01-271-0/+5
| | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | Add decorations to commit pagesJustin Waters2009-01-272-0/+5
| | | | | | | | | | | | | | | | | | | | | This adds the tag and branch head decorations to the commit pages. This is similar to how commits are displayed in the standard gitweb interface. Signed-off-by: Justin Waters <justin.waters@timesys.com>
* | | Merge branch 'og/tree-view-selection'Lars Hjemli2009-01-273-28/+34
|\ \ \ | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | changed objects are outputted, allowing for selections of code onlyOnne Gorter2009-01-103-28/+34
| | | |
* | | | Merge branch 'lh/stats'Lars Hjemli2009-01-2712-4/+569
|\ \ \ \ | |_|/ / |/| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Conflicts: cgit.c cgit.css cgit.h ui-tree.c Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | ui-stats.c: reuse cgit_add_hidden_formfields()Lars Hjemli2008-12-071-3/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This makes sure that the currect branch and path is preserved when changing the stats period or author count. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | ui-shared: externalize add_hidden_formfields()Lars Hjemli2008-12-072-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | This function will be reused by ui-stats.c in the next commit. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | ui-tree: add link to stats page per pathLars Hjemli2008-12-071-0/+3
| | | | | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | ui-shared: add and use cgit_stats_link()Lars Hjemli2008-12-072-2/+10
| | | | | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | ui-stats: replace 'enable-stats' setting with 'max-stats'Lars Hjemli2008-12-078-58/+104
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The new 'max-stats' and 'repo.max-stats' settings makes it possible to define the maximum statistics period, both globally and per repo. Hence, it is now feasible to allow statistics on repositories with a high commit frequency, like linux-2.6, by setting repo.max-stats to e.g. 'month'. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | ui-stats: enable path-filtered statsLars Hjemli2008-12-071-3/+15
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When a path is specified on the querystring the commit statistics will now be filtered by this path. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | Add a 'stats' page to each repoLars Hjemli2008-12-0610-0/+497
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | This new page, which is disabled by default, can be used to print some statistics about the number of commits per period in the repository, where period can be either weeks, months, quarters or years. The function can be activated globally by setting 'enable-stats=1' in cgitrc and disabled for individual repos by setting 'repo.enable-stats=0'. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | | | Merge branch 'stable'Lars Hjemli2009-01-122-8/+11
|\ \ \ \ | | |_|/ | |/| |
| * | | Makefile: install cgit.{css,png} in CGIT_DATA_PATHTodd Zullinger2009-01-121-4/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | CGIT_DATA_PATH defaults to CGIT_SCRIPT_PATH, but allows users to install the cgi and the data files in different locations. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | Makefile: add INSTALL var to set install commandTodd Zullinger2009-01-121-4/+5
| | | | | | | | | | | | | | | | | | | | | | | | | | | | This also explicitly sets the modes for installed files (755 for the .cgi, 644 for the .css and .png). Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | Fix tar.bz2 snapshot exampleTodd Zullinger2009-01-121-2/+2
| | | | | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | | | Merge branch 'stable'Lars Hjemli2009-01-112-2/+6
|\| | |
| * | | Avoid SEGFAULT on invalid requestsLars Hjemli2009-01-112-2/+6
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | When an unknown page is requested, either on the querystring or via PATH_INFO, we end up with a null-referencing cgit_cmd. This null- pointer is then used as argument to the hc() function (which decides what tab to render as 'active'), but this function failed to check if a valid cmd was specified and a SEGFAULT would occur. This patch fixes the issue by introducing a 'fallback-cmd' which specifies what tab to render as 'active' when no valid cmd is requested. While at it, we now also keep track of the active repository even if an invalid cmd was requested since we want to show the error message about the invalid request in the correct context. Noticed-by: Robin Redeker <elmex@ta-sa.org> Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | | | Make all tags viewableRobin Redeker2009-01-112-2/+11
| | | | | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | | | Merge branch 'stable'Lars Hjemli2009-01-111-2/+3
|\| | |
| * | | ui-tag: escape tagnames properlyLars Hjemli2009-01-111-2/+3
| | | | | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | | | Change toggle to more meaningful termJustin Waters2009-01-111-1/+2
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | In the log interface, there is a toggle link at the top, but it isn't clear what's being toggled. I've changed it to "Expand" and "Collapse" to make it clear that you are getting more and less information, respectively. Signed-off-by: Justin Waters <justin.waters@timesys.com>
* | | | Merge branch 'snapshot-fixes'Lars Hjemli2009-01-112-13/+15
|\ \ \ \
| * | | | tests/t0107-snapshot.sh: make testscript match updated snapshot behaviorLars Hjemli2008-12-281-10/+10
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Since cgit now returns a 404 error when the snapshot filename cannot be resolved to a commit object, the testscript needs to request a valid snapshot name. Also, the script assumed that the toplevel directory in the snapshot would get the name of the repository but it's now named similar to the requested snapshot. Signed-off-by: Lars Hjemli <hjemli@gmail.com>
| * | | | return 404 if snapshot is not foundNatanael Copa2008-12-281-3/+5
| | | | | | | | | | | | | | | | | | | | Signed-off-by: Lars Hjemli <hjemli@gmail.com>
* | | | | Merge branch 'lh/mimetypes'Lars Hjemli2009-01-112-3/+3
|\ \ \ \ \
| * | | | | ui-snapshot.c: change mime-type for tar.gz and tar.bz2Lars Hjemli2008-12-262-3/+3
| |/ / / / | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The updated mime-types seems to work better than the old ones with both safari and firefox. Signed-off-by: Lars Hjemli <hjemli@gmail.com>