| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
| |
dirp:iterate()'s fileinfo may not have d_type initialised,
do the check could fail even if it was a valid plugin file.
I think the filename check is sufficient.
|
|
|
|
|
|
|
|
|
| |
This fixes a bug that causes gitano to crash when it's given an empty
command list
Opening and closing of the log should be done in the application,
so it can log before checking for authorization, log after, and
clean up at the end
|
| |
|
| |
|
| |
|
| |
|
|
|
|
| |
Signed-off-by: Vanush "Misha" Paturyan <misha@cs.nuim.ie>
|
|
|
|
|
| |
At the moment the test suite requires that certain errors
produce "Unable to continue" on stderr
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
|
| |
Each of these callsites independently detected repositories before. Now they
use the configurable repository detection which means (a) they are unified in
their behaviour and (b) they can take advantage of new commands which might not
have repositories represented in the same way.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
|
|
|
|
|
|
|
|
|
| |
Up until now, all commands have been restricted to expecting the repository as
the second argument *ONLY*. In order to better support other styles of
command, this routine combines the behaviour of all places which currently look
for the repository into one place. In addition, this means we can simplify
several call sites.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
|
|
|
|
|
|
|
|
| |
This patch adds support for Gitano to load plugins from a colon separated set
of paths. While colon separation is not the best mechanism, it is in common
use. We explicitly do not add any escaping rules such as :: -> : in order to
remain compatible with things like PATH processing.
Signed-off-by: Daniel Silverstone <daniel.silverstone@codethink.co.uk>
|
| |
|
| |
|
| |
|
|
|
|
|
| |
We now perform auth in 2 cgis as well as gitano-auth,
so auth needs to be in the library
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
| |
|
|
|
|
|
|
|
|
| |
Unfortunately the fix to prevent assert()s in the HTTP client would instead
result in silent death in all cases.
This fixes that, by correctly locating the response in the 'ok' variable, ready
for matching.
|
| |
|
|\
| |
| |
| | |
Reviewed-By: Daniel Silverstone <dsilvers@digital-scurf.org>
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This attempts to create the target repository outside the call to
util.copy_dir(), since internally it uses mkdir_p, which doesn't
distinguish between a repository that already exists.
If this check is not made, then it will fail to create a file inside
the repository since it already exists, then attempt to clean up after
itself, possibly silently breaking the in progress copy.
|
| |
| |
| |
| |
| | |
This will create a repository in a different path, so it does not
appear until the copy is complete.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This is complicated enough it could be split out into its own module.
The most important function is copy_dir, which along with the source
and target, is given an optional table of functions, which can be
used to change its behaviour for different types of files.
It may also be given a filter function to determine which files to
process. The default function excludes . and .. of each directory
from the set of directories to process.
It is given the parent directory, filename and fileinfo of every
file to process, so it may exclude based on any combination of
location, file name and file type by returning true.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
hardlink file converts from luxio's raw interface to
boolean success and error message return values, which are used
elsewhere in the codebase.
copy_symlink creates a symlink with the same target as the source
symlink.
|
| | |
|
| |
| |
| |
| |
| | |
It's longer in these cases, but it's more explicitly creating
a path when using a function for it.
|
| |
| |
| |
| |
| |
| |
| |
| | |
This is like python's os.path.join. It is shorter to write
`util.path_join("foo", "bar", "baz")` than
`table.concat({"foo", "bar", "baz"}, "/")`, and I think it looks nicer.
If any special path handling is later required, then it can go there.
|
| | |
|
| |
| |
| |
| | |
Add to the readme command the ability to alter the README of a repository.
|
| |
| |
| |
| |
| | |
Add a set_readme method to gitano.repository so that we can programmatically
change the README.
|
| |
| |
| |
| |
| | |
This adds reasons to all generate_confirmation calls which were
still lacking them.
|
| |
| |
| |
| |
| |
| | |
Before this patch, if you used 'group del' to remove a group
which was a subgroup member of another group, the resulting config
would not parse. This fixes that.
|