| Age | Commit message (Collapse) | Author |
|---|
|
git prefers $VISUAL to $EDITOR. Most of the time. Fortunately we can
just ask git what editor to use rather than trying to figure it out
ourselves.
https://bugzilla.gnome.org/show_bug.cgi?id=654396
|
|
The cookies were being pulled out of Sqlite were Unicode, which
caused the entire HTTP buffer to be Unicode. When concatenating
non-ascii string instances (as would be returned from e.g. git),
things blew up.
Ensure the cookie data is UTF-8 str, and also set our content
encoding explicitly to UTF-8.
https://bugzilla.gnome.org/show_bug.cgi?id=633729
|
|
Should have been 'git commit --amend'. Noticed by Matěj Cepl.
|
|
In Python 2.7 xmlrpclib was changed to use httplib.HTTP[S]Connection
instead of the deprecatede httplib.HTTP[S]. This broke our
xmlrpclib.Transport subclass which overrode make_connection().
For compatibility with old and new Python, switch to subclassing
Transport and SafeTransport separately, with a mixin to add cookies.
https://bugzilla.gnome.org/show_bug.cgi?id=628731
|
|
Allow specifying the default value for whether URLs are added via
a new git config variable. The -u/--add-url and -n/--no-add-url
override this value.
Based on a patch by Xavier Claessens <xclaesse@gmail.com>
|
|
using commit.subject
By mistake, the code was discarding the description after parsing it out from
the edit buffer.
https://bugzilla.gnome.org/show_bug.cgi?id=628455
|
|
In git_run, interactive was initialized twice. Fix from
Felipe Contreras <felipe.contreras@gmail.com>.
|
|
Add documentation for 'path', 'auth-user', and 'auth-password'
|
|
Instead of using authuser and authpwd, spell out auth_user and
auth_password (as was already done in some places.) This changes
the config options to auth-user and auth-password.
|
|
Code to handle parsing a base-path out of the URL would get confused
when given an URL without a path or an URL without a hostname. Catch
those cases.
|
|
|
|
Since Python 2.5, pysqlite2 has been included in the Python standard
library with the name sqlite3.
https://bugzilla.gnome.org/show_bug.cgi?id=621715
|
|
The example url in the section "Ways to refer to a bug" was causing
asciidoc to emit invalid xml (by closing a docbook ulink tag before the
closing semicolon for a > replacement giving ></ulink>;) which
meant that xmlto could not then produce a man page. Quoting the url
seems to workaround this issue.
asciidoc version was 8.4.4
https://bugzilla.gnome.org/show_bug.cgi?id=621580
|
|
|
|
Rather than taking empty or unknown input as a "no", just
reprompt. I don't think the default value is obvious enough
for people to actually figure it out and rely on it, so we might
as well add extra robustness to accidental input.
|
|
Python adds a space character between consecutive calls to
print when the first print doesn't end in a newline. To
avoid problems with this, use <file>.write() rather than
print when:
* We might be printing something empty
* The newline after the line is added via echoed user
input on the terminal
Tracked down by William Jon McCann <william.jon.mccann@gmail.com>
|
|
|
|
Chromium and Google Chrome have the same basic way of storing
cookes, but different config paths. Add a google-chrome browser
option with the correct config path for Google Chrome.
Based on a patch by Eitan Isaacson <eitan@monotonous.org>
|
|
When they moved to webkit as the browser engine, epiphany changed the location
of their cookies database, but the format remained the same. This patch adds
support for the new location, but falls back to the old location if the new one
doesn't exist.
https://bugzilla.gnome.org/show_bug.cgi?id=609668
|
|
The code that was supposed to handle
'See Mozilla bug http://bugzilla.mozilla.org/show_bug.cgi?...'
as a cross reference was also detecting
'See
https://developer.mozilla.org/en/SpiderMonkey/JSAPI_Reference/JS_MaybeGC
https://bugzilla.gnome.org/show_bug.cgi?id=614725'
As a cross-reference. Fix by requiring the "intervening words" not
to have a : or / in them, since "See <url>" is clearly standalone
and something with a : or a / in it is probably an URL.
Reported by Adel Gadllah
https://bugzilla.gnome.org/show_bug.cgi?id=619328
|
|
When we check for uncommitted changes prior to adding URLs to
to commit messages we should ignore submodules, since they don't
affect the rebase xprocess we use to add URLs.
Reported by Benjamin Otte
https://bugzilla.gnome.org/show_bug.cgi?id=611693
|
|
Expiry times are in microseconds since the epoch (with the epoch
depending on the exact version of Chromium.)
We weren't handling this at all and just considering all cookies
non-expired because of the seconds vs. microseconds difference.
|
|
'git bz add-url' was dying because the code was expecting all subcommands
to have an --add-url option but add-url doesn't.
|
|
Adding an URL to just some commits was broken in the rewrite to
be more like git rebase, shuffle things around to fix that.
|
|
I had accidentally prepended the bugzilla installation path in both
BugServer.send_post() and BugServer.send_request() (which is called by
send_post()), so the path was getting prepended twice when trying to attach a
patch.
|
|
In BugHandle.parse(), we first try to parse the bug reference as a url and then
fall back to interpreting it as an alias from the config file. Unfortunately,
something like "gnome:123456" gets parsed as a url (with a scheme of 'gnome' and
a path of '123456', and no hostname). This resulted in us passing a None
hostname to tracker_get_auth_user(), which resulted in an uncaught exception.
From now on, only proceed with treating the bug reference as a url if the scheme
is parsed as 'http' or 'https'.
|
|
This patch should support http authentication credentials specified in url form
(e.g. https://user:password@foo.com) or in a configuration file. The additional
configure options are:
- bz-tracker.<alias>.authuser
- bz-tracker.<alias>.authpwd
|
|
Currently, git-bz assumes that all bugzilla installations are at the root of the
host (e.g. http://foo.com/). It is currently impossible to use git-bz with a
bugzilla installation that is hosted at e.g. foo.com/bugzilla/. This patch adds
that ability. A new configuration option ('bz-tracker.alias.path') allows the
path to be specified for a host.
|
|
bugzilla.gnome.org now supports and redirects to https; even though we
handle the redirects, having it as https in the config will save
a round trip.
|
|
Handle redirect HTTP responses, in particular if a Bugzilla server
is redirecting from http to https.
We try to detect "Bugzilla URL base is over here" when we ask for
show_bug.cgi and remember that for future requests to the same BugServer
to avoid too many redirections.
Switch from caching connection on the BugServer to a global connection
cache, and rewrite the BugServer cache so to deal with the possibility
of redirections.
|
|
I'm not entirely up to speed with the new syntax of Python 2.6, but
apparently you can now do:
func(*args, named_arg=value, **options)
and:
func(*args, named_arg=value)
and Python will happily coalesce named_arg into **options for you.
This is not a valid syntax for Python 2.5, though.
|
|
Add one-stop-shopping --fix=<bug reference> options to 'git bz push'
and 'git bz edit'. These combine attaching the patches and resolving
the bug for cases where review and testing aren't necessary.
|
|
Instead of resetting the branch and applying the patches there,
move to a detached HEAD, apply patches to the detached HEAD and
then make a single update to the branch ref.
The only real advantage of this is that 'git reflog show <branchname>'
shows a single meaningful commit.
|
|
- Always do 'git bz attach --edit'
- Apply patches as a single mailbox
- Make -u/--add-url kinder on the reflog
|
|
Move the documentation from a giant comment at the start of the file
into a separate asciidoc document. Among other revisions an
'EXAMPLE SESSION' section is added to give a sense of the normal
workflow.
Add a Makefile and asciidoc.conf for converting the asciidoc into HTML
or a man page.
|
|
- Remove the -p shortcut for 'git bz edit --pushed' - I don't find
it sufficiently mnemonic to be worth confusion with things like
'git add -p'.
- Say -n/--no-add-url instead of just --no-add-url in an error message.
- Fix usage messages to consistently use () for grouping and [] for
optional parts.
|
|
When there is a single bug referenced in the commits being attached,
allow using that instead of specifying a bug reference manually.
|
|
1) It's confusing to have it different
2) For git bz push, we want to have that association whenever
possible, even if you didn't create the patch yourself.
|
|
If patches have a status and it's 'committed' or 'rejected', then
skip them for git-bz apply and don't prompt.
|
|
Allow configuring the behavior of add-url and --add-url with the
bz.add-url-method config variable. For example:
git config bz.add-url-method subject-prepend:Bug %d -
|
|
* Avoid false positives when the bug # is a substring of a number
in the commit.
* Look at the header, not just the body, for an existing bug #
* When there are local changes, don't print the diffs to the
the screen.
* For 'git bz add-url' indicate what commits we are skipping, and
don't prompt if we aren't going to do anything.
* Don't start the rebase procedure if we aren't going to do anything.
|
|
Switch the default for 'git bz file' and 'git bz attach' so that they
automatically add the URL to the commit message. Add -n/--no-add-url
options to disable adding the URL.
|
|
If we make attachment changes first, we change the timestamp of the
bug; rather than refetching a new token for the bug (or parsing it
out of the HTML response), we just make the bug changes first and
the attachment changes second.
|
|
Fix a problem with adding patches to the 'unapplied_patches' set.
|
|
Use our constant-field-value cache to cache legal field values
for each server; this cuts round trips for 'git bz edit' from
3 to 1, which is quite noticeably faster.
The legal_values method is moved from Bug to BugServer, where
it makes more sense.
|
|
We don't want to constantly refetch things like legal field values
from the server. Add a simple cache based on RawConfigParser and
pickled values stored in ~/.git-bz-cache.
|
|
When listing the available commands, include 'push'
|
|
token = bug.find("token")
self.token = token.text if token else None
Was behaving strangely: apparently ElementTree can be logically
false when non-None. Switch to the more explicit:
self.token = None if token is None else token.text
|
|
Field value for isobsolete must be string "1" not 1.
|
|
* Catch the server returning a 500 protocol error on xmlrpc.cgi;
this happens with bugzilla.openedhand.com and probably other
older versions of post 3.0 bugzilla.
* Handle the server not sending bug/attachment tokens
* Make patch-status handling conditional on the server having
status on patches.
|
