git-bz/.git
4 months agoSigned-off-by: Mason James <mtj@kohaaloha.com> fishsoup
Mason James [Mon, 7 Feb 2022 14:08:45 +0000 (03:08 +1300)]
Signed-off-by: Mason James <mtj@kohaaloha.com>
modified:   git-bz
new file:   git-bz.new
new file:   git-bz.o1
new file:   git-bz.old

4 months agoupdate with new
Mason James [Mon, 7 Feb 2022 13:42:03 +0000 (02:42 +1300)]
update with new

modified:   git-bz

4 months agoadd force attach
Mason James [Mon, 7 Feb 2022 13:34:40 +0000 (02:34 +1300)]
add force attach

modified:   git-bz

6 years agoMerge remote-tracking branch 'kc/fishsoup' into kcfishsoup origin/fishsoup
Chris Cormack [Mon, 13 Jul 2015 04:39:27 +0000 (16:39 +1200)]
Merge remote-tracking branch 'kc/fishsoup' into kcfishsoup

6 years agoFix for git-bz to deal with the fact the output of changing a status has changed...
Chris Cormack [Mon, 13 Jul 2015 04:37:41 +0000 (16:37 +1200)]
Fix for git-bz to deal with the fact the output of changing a status has changed slightly

7 years agoFixing utf-8 encoding issues. Special utf-8 characters can now be used in commit...
Justin [Thu, 15 Jan 2015 15:30:28 +0000 (15:30 +0000)]
Fixing utf-8 encoding issues. Special utf-8 characters can now be used in commit messages. Added a definition header for the source code encoding.

8 years agoteach git-bz how to deal with BZ's CSRF protection
Galen Charlton [Sun, 20 Apr 2014 14:54:25 +0000 (14:54 +0000)]
teach git-bz how to deal with BZ's CSRF protection

This is pretty hackish; it may be time to see if we can
do what we need using BZ's official web services.

Signed-off-by: Galen Charlton <gmc@esilibrary.com>
8 years agoTeach git-bz to update dependson
Hugh Davenport [Sat, 6 Jul 2013 00:34:35 +0000 (12:34 +1200)]
Teach git-bz to update dependson

Uses a commit tag Depends: which needs to map the regexp (bug)?\s*(\d+)
case insensitive. Allows multiple depends lines, and shows the current
setting of the depends.

Signed-off-by: Hugh Davenport <hugh@davenport.net.nz>
Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
8 years agoMerge branch 'master' of git://git.fishsoup.net/git-bz into fishsoup
Jared Camins-Esakov [Wed, 10 Jul 2013 14:39:43 +0000 (10:39 -0400)]
Merge branch 'master' of git://git.fishsoup.net/git-bz into fishsoup

9 years agoDon't choke on UTF-8 chars in bugzilla summaries when writing commit templates.
Trevor Saunders [Tue, 17 Jul 2012 16:50:33 +0000 (12:50 -0400)]
Don't choke on UTF-8 chars in bugzilla summaries when writing commit templates.

Conflicts:
git-bz

9 years agoGet a new token for each attachment.
Bobby Holley [Fri, 2 Dec 2011 21:22:51 +0000 (13:22 -0800)]
Get a new token for each attachment.

9 years agoMerge remote-tracking branch 'kc/fishsoup' into HEAD
Chris Cormack [Mon, 24 Jun 2013 08:06:12 +0000 (20:06 +1200)]
Merge remote-tracking branch 'kc/fishsoup' into HEAD

9 years agoUse the CSRF token from the attachment page, rather than the bug page, when attaching...
Bobby Holley [Wed, 30 Nov 2011 05:33:21 +0000 (21:33 -0800)]
Use the CSRF token from the attachment page, rather than the bug page, when attaching files.

This fixes breakage caused by a recent BMO change.

A similar change was made to bzapi: http://hg.mozilla.org/webtools/bzapi/rev/5ca3562f2e6c

9 years agoFix typo: Uncommment => uncomment
Owen W. Taylor [Thu, 14 Feb 2013 14:50:45 +0000 (09:50 -0500)]
Fix typo: Uncommment => uncomment

Pointed out by Dan Winship

9 years agoMake changing status and patch complexity at the same time work
Jared Camins-Esakov [Fri, 18 Jan 2013 02:49:51 +0000 (21:49 -0500)]
Make changing status and patch complexity at the same time work

9 years agoTeach git-bz to update patch-complexity
Hugh Davenport [Wed, 16 Jan 2013 22:17:00 +0000 (11:17 +1300)]
Teach git-bz to update patch-complexity

Note that this commit hardcodes the complexities relevant for the Koha
project.

Signed-off-by: Jared Camins-Esakov <jcamins@cpbibliography.com>
9 years agoDon't add user to CC list so that updating status will work
Jared Camins-Esakov [Sat, 10 Nov 2012 13:46:14 +0000 (08:46 -0500)]
Don't add user to CC list so that updating status will work

9 years agoAdd a missing declaration
Jared Camins-Esakov [Sat, 10 Nov 2012 13:22:17 +0000 (08:22 -0500)]
Add a missing declaration

9 years agoAdd README to fishsoup
Jared Camins-Esakov [Tue, 6 Nov 2012 20:46:26 +0000 (15:46 -0500)]
Add README to fishsoup

9 years agoFix merge error
Jared Camins-Esakov [Tue, 6 Nov 2012 21:20:45 +0000 (16:20 -0500)]
Fix merge error

9 years agoTeach git-bz to optionally sign off when applying patches
Jared Camins-Esakov [Fri, 2 Nov 2012 12:00:17 +0000 (08:00 -0400)]
Teach git-bz to optionally sign off when applying patches

Adds a -s/--signoff option for the apply action. When set, git-bz
will instruct git-am to sign off on all patches as they are being
applied.

WARNING: THIS OPTION IS INTENDED FOR USE WHEN PUSHING BRANCHES. DO
NOT USE THIS OPTION AS AN ALTERNATIVE TO ACTUALLY TESTING CODE.

Conflicts:

git-bz

9 years agoadd support to email patch to 'koha-patches' mailing-list, using --mail arg
Mason James [Tue, 25 Sep 2012 12:33:50 +0000 (00:33 +1200)]
add support to email patch to 'koha-patches' mailing-list, using --mail arg

Conflicts:

git-bz

9 years agoTeach git-bz to update bug statuses
Jared Camins-Esakov [Sat, 14 Jul 2012 13:52:09 +0000 (09:52 -0400)]
Teach git-bz to update bug statuses

Note that this commit hardcodes the statuses relevant for the Koha
project.

Conflicts:

git-bz

9 years agoAdd support for bz-user and bz-password and authenticating directly.
software.coop worker [Sat, 4 Jun 2011 17:54:18 +0000 (17:54 +0000)]
Add support for bz-user and bz-password and authenticating directly.

Conflicts:

git-bz

9 years agoTypo fix
Chris Cormack [Tue, 24 May 2011 21:46:29 +0000 (09:46 +1200)]
Typo fix

9 years agoFixing a little copy and paste error with auth-user and auth-password
Chris Cormack [Wed, 18 May 2011 08:29:38 +0000 (20:29 +1200)]
Fixing a little copy and paste error with auth-user and auth-password

9 years agoAdd support for epiphany 3.6
Bastien Nocera [Thu, 17 May 2012 14:12:25 +0000 (15:12 +0100)]
Add support for epiphany 3.6

Cookie jar got migrated from ~/.gnome2/epiphany to ~/.config/epiphany

https://bugzilla.gnome.org/show_bug.cgi?id=676241

10 years agoattach: Obsolete patches with the same subject
Jasper St. Pierre [Mon, 12 Mar 2012 20:27:35 +0000 (16:27 -0400)]
attach: Obsolete patches with the same subject

For plain 'git bz attach', always obsolete existing patches if they
have the same subject as the commit being attached.

For 'git bz attach -e', make this the default.

https://bugzilla.gnome.org/show_bug.cgi?id=671941

10 years agoapply: fix typo
Owen W. Taylor [Thu, 1 Mar 2012 20:46:09 +0000 (15:46 -0500)]
apply: fix typo

""" not """"

10 years agoapply: Fix addition of 'need commit message' string
Owen W. Taylor [Thu, 1 Mar 2012 20:35:57 +0000 (15:35 -0500)]
apply: Fix addition of 'need commit message' string

Fix a double addition of the commit message, and add a comment
pointing out that the exact string is checked by git.gnome.org.

10 years agoapply: ask the user to double-check author and subject
Owen W. Taylor [Thu, 1 Mar 2012 20:31:33 +0000 (15:31 -0500)]
apply: ask the user to double-check author and subject

The author and subject might be wrong (especially the subject), when
we make up a commit message for a plain patch, so ask the user to
double check.

10 years agoapply: pass _interactive=True when running 'git commit --amend'
Owen W. Taylor [Thu, 1 Mar 2012 20:30:04 +0000 (15:30 -0500)]
apply: pass _interactive=True when running 'git commit --amend'

If the editor uses the terminal, the missing _interactive=True will
cause things to hang.

10 years agoapply: attempt to handle plain diffs
Dan Winship [Tue, 2 Aug 2011 14:57:48 +0000 (10:57 -0400)]
apply: attempt to handle plain diffs

If a bug has a plain diff (as opposed to git-format-patch output),
prepend some minimal headers to it to get it into a format git-am will
accept, and then make the user write a commit message for it before
proceeding.

https://bugzilla.gnome.org/show_bug.cgi?id=657558

10 years agoapply: add --continue/--skip/--abort
Dan Winship [Sat, 9 Jul 2011 14:56:30 +0000 (10:56 -0400)]
apply: add --continue/--skip/--abort

Rather than failing completely when git-am fails to apply a patch,
write out our current state to git-am's temporary directory, and then
tell the user to use "git bz apply --continue/--skip/--abort" after
handling the merge failure. When the user uses one of those options,
read back our state, pass the flag on to git-am, and (assuming git-am
succeeds), continue with the next patch.

https://bugzilla.gnome.org/show_bug.cgi?id=657558

10 years agoapply: add interactive option
Dan Winship [Sat, 9 Jul 2011 18:10:50 +0000 (14:10 -0400)]
apply: add interactive option

Rather than asking whether or not to apply each patch individually,
list all of the patches at once and allow the user to either apply all
of them, apply none of them, or interactively choose which ones to
apply (a la interactive rebasing).

In addition to choosing which patches to apply, the interactive option
also allows reordering the patches before applying, and choosing to
apply committed or rejected patches.

https://bugzilla.gnome.org/show_bug.cgi?id=657558

10 years agoapply: pass "-3" to git-am
Dan Winship [Sat, 9 Jul 2011 15:01:54 +0000 (11:01 -0400)]
apply: pass "-3" to git-am

Using 3-way merge makes it much more likely that the patch will apply
correctly when the local tree is newer than the tree that the patch
was generated from, and if the patch doesn't apply cleanly, git will
leave a file with conflict markers rather than forcing you to reapply
the patch manually.

https://bugzilla.gnome.org/show_bug.cgi?id=657558

10 years agoWork around line-break problems
Owen W. Taylor [Thu, 1 Mar 2012 16:48:17 +0000 (11:48 -0500)]
Work around line-break problems

The two lines for the different versions of 'git bz edit' are treated
by asciidoc as a single paragraph and line-wrapped.

In theory, a + at the end of a line is a forced line-break with
asciidoc, but in the manpage generation route, this doesn't work
properly - the magic in a2x that is supposed to turn the asciidoc-br
processing instruction into a line break doesn't work. (We aren't
using a2x at the moment for the conversion, but no real reason to
switch if it doesn't work better.)

So, use a [verse] paragraph instead. This results in mis-indentation,
but slightly better than the previous breakage.

https://bugzilla.gnome.org/show_bug.cgi?id=670686

10 years agoFix up documentation generation rules
Owen W. Taylor [Thu, 1 Mar 2012 16:48:09 +0000 (11:48 -0500)]
Fix up documentation generation rules

10 years agoDon't strip leading space from the commit body
Owen W. Taylor [Fri, 10 Feb 2012 18:45:09 +0000 (13:45 -0500)]
Don't strip leading space from the commit body

If we strip leading space from the body, we'll unindent the
first line of the body, which could break aligment.

Based on a patch by Stef Walter <stefw@gnome.org>

10 years agoFix parameter mistake from last patch
Owen W. Taylor [Tue, 7 Feb 2012 18:27:25 +0000 (13:27 -0500)]
Fix parameter mistake from last patch

Pass all the necessary fields to merge_default_fields_from_dict().

10 years agoMake setting default-product, default-component in tracker config work
Owen W. Taylor [Tue, 7 Feb 2012 01:57:01 +0000 (20:57 -0500)]
Make setting default-product, default-component in tracker config work

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".

10 years agoSupport pretty bugzilla links like https://bugzilla.gnome.org/12345
Colin Walters [Thu, 25 Aug 2011 17:59:31 +0000 (13:59 -0400)]
Support pretty bugzilla links like https://bugzilla.gnome.org/12345

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

10 years agoAllow setting bz.default-<field name>
Owen W. Taylor [Sun, 5 Feb 2012 19:37:38 +0000 (14:37 -0500)]
Allow setting bz.default-<field name>

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

10 years agoRead bz.* git config upfront
Owen W. Taylor [Sun, 5 Feb 2012 18:08:01 +0000 (13:08 -0500)]
Read bz.* git config upfront

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>.

10 years agoAdd defaults for bugs.gentoo.org.
Michał Górny [Mon, 19 Sep 2011 23:15:00 +0000 (01:15 +0200)]
Add defaults for bugs.gentoo.org.

10 years agoAdd the user to the cc: on the bug
Owen W. Taylor [Sat, 4 Feb 2012 22:44:12 +0000 (17:44 -0500)]
Add the user to the cc: on the bug

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

10 years agoFix a copy and paste error with auth-user and auth-password
Chris Cormack [Wed, 18 May 2011 08:31:42 +0000 (20:31 +1200)]
Fix a copy and paste error with auth-user and auth-password

10 years agoSpelling correction
Yawar Amin [Sun, 24 Oct 2010 00:24:30 +0000 (20:24 -0400)]
Spelling correction

10 years agoHandle Unicode bug titles correctly
Owen W. Taylor [Sat, 4 Feb 2012 19:04:27 +0000 (14:04 -0500)]
Handle Unicode bug titles correctly

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

10 years agoCatch trying to refile an already filed bug
Owen W. Taylor [Mon, 12 Sep 2011 20:02:49 +0000 (16:02 -0400)]
Catch trying to refile an already filed bug

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

10 years agoMake components work with older bugzilla
Owen W. Taylor [Fri, 9 Sep 2011 22:31:30 +0000 (18:31 -0400)]
Make components work with older bugzilla

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

10 years agocomponents: new command to list the components of a product
Dan Winship [Fri, 15 Jul 2011 18:25:54 +0000 (14:25 -0400)]
components: new command to list the components of a product

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

10 years agoProperly close the template file
Jasper St. Pierre [Tue, 30 Aug 2011 14:45:07 +0000 (10:45 -0400)]
Properly close the template file

https://bugzilla.gnome.org/show_bug.cgi?id=657723

10 years agoAdd galeon browser support
Yanko Kaneti [Tue, 12 Oct 2010 19:23:07 +0000 (22:23 +0300)]
Add galeon browser support

https://bugzilla.gnome.org/show_bug.cgi?id=632006

10 years agoFix choice of editor
Dan Winship [Sat, 9 Jul 2011 17:44:50 +0000 (13:44 -0400)]
Fix choice of editor

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

11 years agoSupport UTF-8 attachments
Colin Walters [Mon, 1 Nov 2010 18:58:26 +0000 (14:58 -0400)]
Support UTF-8 attachments

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

11 years agoFix 'git command --amend' typo in man page
Owen W. Taylor [Wed, 27 Apr 2011 17:26:01 +0000 (13:26 -0400)]
Fix 'git command --amend' typo in man page

Should have been 'git commit --amend'. Noticed by Matěj Cepl.

11 years agoFix python-2.7 compatibility with xmlrpclib
Owen W. Taylor [Wed, 22 Sep 2010 19:54:36 +0000 (15:54 -0400)]
Fix python-2.7 compatibility with xmlrpclib

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

11 years agoAdd bz.add-url config variable
Owen W. Taylor [Mon, 8 Feb 2010 21:10:36 +0000 (22:10 +0100)]
Add bz.add-url config variable

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>

11 years agoUse the actual description entered by the user, rather than unconditionally using...
Bobby Holley [Sat, 14 Aug 2010 23:26:20 +0000 (19:26 -0400)]
Use the actual description entered by the user, rather than unconditionally 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

11 years agoFix duplicate variable setting
Owen W. Taylor [Mon, 6 Sep 2010 23:56:00 +0000 (19:56 -0400)]
Fix duplicate variable setting

In git_run, interactive was initialized twice. Fix from
Felipe Contreras <felipe.contreras@gmail.com>.

11 years agoDocument new per-tracker configuration
Owen W. Taylor [Mon, 6 Sep 2010 23:30:53 +0000 (19:30 -0400)]
Document new per-tracker configuration

Add documentation for 'path', 'auth-user', and 'auth-password'

11 years agoFix inconsistent authuser/authpwd abbreviations
Owen W. Taylor [Mon, 6 Sep 2010 23:07:35 +0000 (19:07 -0400)]
Fix inconsistent authuser/authpwd abbreviations

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.

11 years agoHandle partial URLs
Owen W. Taylor [Mon, 6 Sep 2010 23:05:41 +0000 (19:05 -0400)]
Handle partial URLs

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.

11 years agoMerge remote branch 'jjongsma/http-auth'
Owen W. Taylor [Mon, 6 Sep 2010 22:48:58 +0000 (18:48 -0400)]
Merge remote branch 'jjongsma/http-auth'

11 years agoUse sqlite3 module if available
Yaakov Selkowitz [Wed, 16 Jun 2010 01:42:12 +0000 (20:42 -0500)]
Use sqlite3 module if available

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

12 years agodocs: Quote example url to workaround asciidoc bug
Robert Bragg [Mon, 14 Jun 2010 18:22:27 +0000 (19:22 +0100)]
docs: Quote example url to workaround asciidoc bug

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 &gt; replacement giving &gt</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

12 years agoFix typo in google-chrome and chromium support
Marc-Andre Lureau [Sun, 6 Jun 2010 13:38:04 +0000 (15:38 +0200)]
Fix typo in google-chrome and chromium support

12 years agoWhen prompting, loop, don't default on unknown inputs
Owen W. Taylor [Sat, 5 Jun 2010 18:54:40 +0000 (14:54 -0400)]
When prompting, loop, don't default on unknown inputs

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.

12 years agoFix problem with stray spaces
Owen W. Taylor [Sat, 5 Jun 2010 18:30:56 +0000 (14:30 -0400)]
Fix problem with stray spaces

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>

12 years agoFix typos in documentation
Christophe Fergeau [Thu, 22 Oct 2009 18:48:47 +0000 (20:48 +0200)]
Fix typos in documentation

12 years agoAdded google-chrome as a possible browser
Owen W. Taylor [Wed, 17 Feb 2010 20:30:24 +0000 (12:30 -0800)]
Added google-chrome as a possible browser

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>

12 years agoAdd support for epiphany with the webkit backend
Jonathon Jongsma [Thu, 13 May 2010 16:33:05 +0000 (11:33 -0500)]
Add support for epiphany with the webkit backend

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

12 years agoHandle 'see <url> <bug reference>'
Owen W. Taylor [Sat, 5 Jun 2010 14:27:43 +0000 (10:27 -0400)]
Handle 'see <url> <bug reference>'

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

12 years agoPass --ignore-submodules when checking uncommitted changes
Owen W. Taylor [Sat, 5 Jun 2010 14:08:36 +0000 (10:08 -0400)]
Pass --ignore-submodules when checking uncommitted changes

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

12 years agoHandle Chromium cookie expiry properly
Owen W. Taylor [Sat, 17 Oct 2009 15:30:28 +0000 (11:30 -0400)]
Handle Chromium cookie expiry properly

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.

12 years agoFix subcommands without --add-url option
Owen W. Taylor [Fri, 18 Sep 2009 16:54:43 +0000 (12:54 -0400)]
Fix subcommands without --add-url option

'git bz add-url'  was dying because the code was expecting all subcommands
to have an --add-url option but add-url doesn't.

12 years agoFix adding-url to just some commits
Owen W. Taylor [Fri, 18 Sep 2009 16:52:45 +0000 (12:52 -0400)]
Fix adding-url to just some commits

Adding an URL to just some commits was broken in the rewrite to
be more like git rebase, shuffle things around to fix that.

12 years agoFix ' git bz attach' failure when bugzilla server has a path
Jonathon Jongsma [Tue, 15 Sep 2009 20:01:05 +0000 (15:01 -0500)]
Fix ' git bz attach' failure when bugzilla server has a path

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.

12 years agoFix exception when parsing an alias as a url
Jonathon Jongsma [Tue, 15 Sep 2009 19:54:02 +0000 (14:54 -0500)]
Fix exception when parsing an alias as a url

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'.

12 years agoAdd support for bugzilla installations that require http auth
Jonathon Jongsma [Fri, 11 Sep 2009 04:35:25 +0000 (23:35 -0500)]
Add support for bugzilla installations that require http auth

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

12 years agoAdd support for bugzilla installations in non-root paths
Jonathon Jongsma [Fri, 11 Sep 2009 04:01:20 +0000 (23:01 -0500)]
Add support for bugzilla installations in non-root paths

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.

12 years agoMark bugzilla.gnome.org as https
Owen W. Taylor [Wed, 9 Sep 2009 21:21:43 +0000 (17:21 -0400)]
Mark bugzilla.gnome.org as https

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.

12 years agoHandle redirects
Owen W. Taylor [Wed, 9 Sep 2009 21:14:55 +0000 (17:14 -0400)]
Handle redirects

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.

12 years agoAvoid Python2.6-isms
Emmanuele Bassi [Wed, 9 Sep 2009 14:06:37 +0000 (15:06 +0100)]
Avoid Python2.6-isms

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.

12 years agoAdd 'git bz push -fix' and 'git bz edit --fix'
Owen W. Taylor [Sat, 5 Sep 2009 17:19:11 +0000 (13:19 -0400)]
Add 'git bz push -fix' and 'git bz edit --fix'

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.

12 years agoMake 'git bz add-url' more like git rebase
Owen W. Taylor [Sat, 5 Sep 2009 15:30:10 +0000 (11:30 -0400)]
Make 'git bz add-url' more like git rebase

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.

12 years agoTODO updates
Owen W. Taylor [Sat, 5 Sep 2009 14:10:43 +0000 (10:10 -0400)]
TODO updates

 - Always do 'git bz attach --edit'
 - Apply patches as a single mailbox
 - Make -u/--add-url kinder on the reflog

12 years agoSplit out documentation
Owen W. Taylor [Sat, 5 Sep 2009 13:50:38 +0000 (09:50 -0400)]
Split out documentation

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.

12 years agoCommand line option tweaks
Owen W. Taylor [Sat, 5 Sep 2009 13:27:51 +0000 (09:27 -0400)]
Command line option tweaks

- 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.

12 years agoAllow omitting bug reference for 'git bz attach'
Owen W. Taylor [Sat, 5 Sep 2009 13:06:19 +0000 (09:06 -0400)]
Allow omitting bug reference for 'git bz attach'

When there is a single bug referenced in the commits being attached,
allow using that instead of specifying a bug reference manually.

12 years agoMake --add-url the default for 'git bz apply' as well
Owen W. Taylor [Sat, 5 Sep 2009 00:55:54 +0000 (20:55 -0400)]
Make --add-url the default for 'git bz apply' as well

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.

12 years agoIgnore committed/rejected patches
Owen W. Taylor [Sat, 5 Sep 2009 00:39:10 +0000 (20:39 -0400)]
Ignore committed/rejected patches

If patches have a status and it's 'committed' or 'rejected', then
skip them for git-bz apply and don't prompt.

12 years agoAllow configuring bug reference addition
Owen W. Taylor [Wed, 2 Sep 2009 00:01:40 +0000 (20:01 -0400)]
Allow configuring bug reference addition

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 -

12 years agoImprove URL addition
Owen W. Taylor [Tue, 1 Sep 2009 23:13:34 +0000 (19:13 -0400)]
Improve URL addition

* 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.

12 years agoAdd URLs by default
Owen W. Taylor [Tue, 1 Sep 2009 04:17:06 +0000 (00:17 -0400)]
Add URLs by default

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.

12 years agoMake bug changes before attachment changes to avoid token problems
Owen W. Taylor [Tue, 1 Sep 2009 03:54:49 +0000 (23:54 -0400)]
Make bug changes before attachment changes to avoid token problems

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.

12 years agoUse .add() for sets not .append()
Owen W. Taylor [Mon, 31 Aug 2009 17:53:59 +0000 (13:53 -0400)]
Use .add() for sets not .append()

Fix a problem with adding patches to the 'unapplied_patches' set.

12 years agoCache legal field values
Owen W. Taylor [Mon, 31 Aug 2009 05:06:21 +0000 (01:06 -0400)]
Cache legal field values

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.

12 years agoAdd utility for constant-response caching
Owen W. Taylor [Mon, 31 Aug 2009 05:05:07 +0000 (01:05 -0400)]
Add utility for constant-response caching

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.