Age | Commit message (Collapse) | Author |
|
Allow setting the default-product in the tracker config. This is possibly
useful for something like bugzilla.redhat.com where many repositories share
a product of "Fedora".
|
|
Ideally I'd like git-bz to know when a Bugzilla instance supports
these, but here at least we'll try parsing them.
https://bugzilla.gnome.org/show_bug.cgi?id=657361
|
|
If you want to override something like the default version on a per-component
basis, having to set that in the per-tracker configuration is unnatural
and confusing. Allow setting bz.default-version instead. This is interpreted
as overriding the existing configuration of default field values.
Based on a patch from Dan Winship <danw@gnome.org>
https://bugzilla.gnome.org/show_bug.cgi?id=651756
|
|
Instead of making calls to git.config() as needed, read all of our options
ahead of time into a dictionary initialized with defaults.
Loosely based on a patch from Felipe Contreras <felipe.contreras@gmail.com>.
|
|
|
|
All the logic to add the user to the cc: list on the bug is done in the
web page for show_bug.cgi and not on the client side, so the option
"Automatically add me to the CC list of bugs I change" in the user's
preferences is not honored. Since that preference defaults to Always
and users will very typically want to see further traffic on their patches,
just always request the user to be added to the Cc.
The main downside to this is that we do another round trip to the server
when creating and attachment, and also potentially create more bug mail.
https://bugzilla.gnome.org/show_bug.cgi?id=663800
|
|
|
|
|
|
If the data that we are writing into the template ends includes
non-ASCII data, it will need to be encoded as UTF-8 before writing
the template file. Fix from Luca Bruno.
https://bugzilla.gnome.org/show_bug.cgi?id=657716
|
|
If 'git bz file' is used with a commit message that contains a
bug reference, prompt before filing again.
https://bugzilla.gnome.org/show_bug.cgi?id=624461
|
|
Use Bug.legal_values({ product_id: <id> }) to find out legal
component values in a way that works on older Bugzilla like the
bugzilla.gnome.org 3.4.
This requires finding out the product_id first, which is fast
with new bugzilla, but requires a slow fallback on older bugzillas;
we cache positive hits for product IDs in our cache.
https://bugzilla.gnome.org/show_bug.cgi?id=654693
|
|
eg
git bz components -b bugzilla.gnome.org gnome-shell
or just
git bz components
if you have defaults configured
https://bugzilla.gnome.org/show_bug.cgi?id=654693
|
|
https://bugzilla.gnome.org/show_bug.cgi?id=657723
|
|
https://bugzilla.gnome.org/show_bug.cgi?id=632006
|
|
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.
|