| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
|
|
|
|
| |
Current script was choosing only automatically default profile for the
cookiefile to be used. Unfortunately, when the user doesn't explicitly
set default profile, Firefox sets as default the last profile used,
which can make git-bz choosing the wrong profile.
This patch makes git-bz to make the choice explicit and ask in the
dialog for the corrent profile.
https://bugzilla.gnome.org/show_bug.cgi?id=658539
|
|
|
|
|
| |
(Reconstructed from the original commit
http://git.fishsoup.net/cgit/git-bz/commit/?id=e17bbae7a2ce454d9f69c32fc40066995d44913d)
|
|
|
|
|
|
|
|
| |
Chrome and Chromium now have cookies stored in their sqlite database
encrypted, there's a new column 'encrypted_value' in the cookies
table. See also https://codereview.chromium.org/24734007
https://bugzilla.gnome.org/show_bug.cgi?id=746137
|
|
|
|
|
|
|
|
| |
Reference https://www.python.org/dev/peps/pep-0394/ - although /usr/bin/python
is SUPPOSED to be python2, apps that depend on python2 should specify that
explicitly to accomodate Arch Linux and the future.
https://bugzilla.gnome.org/show_bug.cgi?id=746071
|
|
|
|
|
|
|
| |
bugzilla.mozilla.org is configured so that the title is just "123456"
and it just says "123456 processed"; it would also be possible to
configure a bugzilla instance to say "Issue" (etc.) instead of "Bug" -
try to handle these differences.
|
|
|
|
|
|
|
|
| |
Add the attachment flags that git-bz-moz supports for bugzilla.mozilla.org;
there are more attachment flags available, but listing only the most
important ones reduces noise.
https://bugzilla.gnome.org/show_bug.cgi?id=602406
|
|
|
|
|
|
| |
A multiplicable flag can be requested of several users.
https://bugzilla.gnome.org/show_bug.cgi?id=602406
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
One has to first define the available flags for the bugtracker
used. The only way I found is to look at the source of the bugtracker
"Create New Attachment" page.
Examples of use of a review: flag below. Self-review would be:
review: +
Requesting a review from a specific person would look like:
review: ? joe@example.com
To request review from a specific person as shown above, the
requesteeable git-bz option of the flag has to be set to true like:
git config bz-tracker.bugzilla.mozilla.org.attachment-flag.review.requesteeable
true
(Patch includes fixes by Owen Taylor <otaylor@redhat.com>)
https://bugzilla.gnome.org/show_bug.cgi?id=602406
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
This improves validation of attach arguments. Previously, the command:
git bz attach 741345
would result in the crash:
fatal: ambiguous argument '741345': unknown revision or path not in
the working tree.
Use '--' to separate paths from revisions, like this:
'git <command> [<revision>...] -- [<file>...]'
Traceback (most recent call last):
File "/home/philip/.local/bin/git-bz", line 2472, in <module>
do_attach(*args)
File "/home/philip/.local/bin/git-bz", line 1830, in do_attach
commits = get_commits(commit_or_revision_range)
File "/home/philip/.local/bin/git-bz", line 233, in get_commits
commits = rev_list_commits(commit_or_revision_range)
File "/home/philip/.local/bin/git-bz", line 205, in
rev_list_commits
output = git.rev_list(*args, **kwargs_copy)
File "/home/philip/.local/bin/git-bz", line 192, in f
return git_run(command, *args, **kwargs)
File "/home/philip/.local/bin/git-bz", line 174, in git_run
raise CalledProcessError(process.returncode, " ".join(to_run))
subprocess.CalledProcessError: Command 'git rev-list
--pretty=format:%s 741345' returned non-zero exit status 128
Now it prints an error gracefully.
https://bugzilla.gnome.org/show_bug.cgi?id=742371
|
|
|
|
|
|
|
|
|
|
| |
Modify cookie getter functions to find the directories where the browser
cookies are stored on Mac OS X.
Applied with minor changes. See also:
http://www.chromium.org/user-experience/user-data-directory
https://bugzilla.gnome.org/show_bug.cgi?id=738337
|
|
|
|
|
|
|
|
|
| |
We ordered status and description differently in the initial display
and in interactive mode. While the two modes are arguably a bit
different in how you are using the information, it's better to use
a consistent ordering so the user can learn where to look.
https://bugzilla.gnome.org/show_bug.cgi?id=678152
|
|
|
|
|
|
|
| |
It's a bit confusing to show some patch statuses, but not others. Switch
to showing all statuses.
https://bugzilla.gnome.org/review?bug=678152
|
|
|
|
|
|
|
|
|
|
| |
When dealing with a patch series attached to one bug, it's hard to
know which of the attachments have been marked as "accepted-commit_now"
or "accepted-commit_after_freeze". This patch shows those statuses
in the original apply question, and in the interactive application
questions.
https://bugzilla.gnome.org/show_bug.cgi?id=678152
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
A non-zero exit status from 'git push' doesn't necessarily mean that
nothing was pushed - we should parse the output just as for a zero
exit status. This requires adding a new option _ignore_exit_status
to git_run().
For --dry, we can actually pay attention to the return code, but need
to fix up previous logic where we returned None instead of a list of
commits, and then tried to iterate over None. For clarity, let
CalledProcessError throw through rather than special-casing a return
of None.
Inspired by a patch from Philip Withnall.
https://bugzilla.gnome.org/show_bug.cgi?id=620668
|
|
|
|
|
|
|
|
| |
If an URL ends up with a unicode type, then that will force the
body of the HTTP request to unicode; adding the raw bytes of the
commit can then cause an encoding error.
https://bugzilla.gnome.org/show_bug.cgi?id=745186
|
|
|
|
|
|
|
| |
Add documentaton of recognized forms of bug references in commit messages,
and explain the way that "See" suppresses bug reference identification.
https://bugzilla.gnome.org/show_bug.cgi?id=708158
|
|
|
|
|
| |
Though it didn't matter here, s.find(x) >= 0 is is better to write
than s.find(x) > 0.
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=673828
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
Specify that bugzilla.redhat.com is https - filing still won't work
without specifying a default-version but we'll get a clean error
message since it will now go through the XML-RPC interface
and not be broken by the redirect (see bug 745100)
Since the version on bugzilla.redhat.com is product specific,
and we can't guess what products people will file bugs against,
specifying a default version doesn't make sense.
https://bugzilla.gnome.org/show_bug.cgi?id=673828
|
|
|
|
|
| |
Existing docs made it a bit hard to figure out how --fix works - list
the usage explicitly.
|
|
|
|
|
|
|
| |
bugzilla.gnome.org now supports attachment status on
attachment.cgi?action=insert.
https://bugzilla.gnome.org/show_bug.cgi?id=745062
|
|
|
|
|
|
| |
Handle the changed name of the attachment status field; status on creation
isn't supported with the current b.g.o code, so leave that as a FIXME
for now.
|
|
|
|
|
|
|
| |
create_patch() now gets attachment.cgi to get a token - with the
switch urllib2 this requires slightly different code.
https://bugzilla.gnome.org/show_bug.cgi?id=725730
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
At times, we can make many separate requests. If we start off with
a URL that bugzilla always redirects (say a HTTP URL), it's better to
"get the message" and start using the target of the redirects.
The code that used to do this was removed with the switch to urllib2.
Add back equivalent code - it can be simplified at this point because
bugzilla was changed to use 301 permanent redirects for HTTP => HTTPS
redirects, and left using 302 requests for attachment
redirects.
https://bugzilla.gnome.org/show_bug.cgi?id=725730
|
|
|
|
|
|
|
| |
This simplifies the code and will auto-detect the users proxy
settings and use those.
https://bugzilla.gnome.org/show_bug.cgi?id=725730
|
|
|
|
|
|
| |
So the cache file will now typically be ~/.cache/git-bz-cache.
https://bugzilla.gnome.org/show_bug.cgi?id=674020
|
|
|
|
|
|
|
|
|
|
| |
By treating all strings internally as Unicode (decoding them on input
and encoding them on output), git commit messages which contain Unicode
can be handled without getting UnicodeDecodeErrors.
This works on Python 2 and 3.
https://bugzilla.gnome.org/show_bug.cgi?id=684578
|
|
|
|
|
|
|
| |
Although none of the code or strings need it at the moment, it’s best
to be explicit.
https://bugzilla.gnome.org/show_bug.cgi?id=684578
|
|
|
|
|
|
|
| |
As a follow up to commit 68256107, change the default platform for GNOME
Bugzilla to the empty string, and ignore empty platform fields when
uploading. This makes Bugzilla use the default value, which would be
‘Other’ or ‘All’ or whatever the product has set up.
|
|
|
|
|
|
|
|
| |
It was previously ‘All’, but this seems to have been removed for all
products with the upgrade to Bugzilla 4.4. Change the default to
‘Other’, which is still present.
https://bugzilla.gnome.org/show_bug.cgi?id=744319
|
|
|
|
| |
https://bugzilla.gnome.org/show_bug.cgi?id=721962
|
|
|
|
|
|
| |
At least bugzilla.redhat.com ones are stored like that.
https://bugzilla.gnome.org/show_bug.cgi?id=673826
|
|
|
|
|
|
| |
So that people know where to report git-bz bugs.
https://bugzilla.gnome.org/show_bug.cgi?id=742370
|
|
|
|
|
|
|
| |
Title it ‘AUTHORS’ rather than ‘Author’ to make it more in line with
other man pages.
https://bugzilla.gnome.org/show_bug.cgi?id=742370
|
|
|
|
|
|
|
|
|
|
| |
attachment.cgi requires a token field for Bugzilla 4.2 and later.
(See https://bugzilla.mozilla.org/show_bug.cgi?id=703983)
This token is not the same as the overall bug's token. So from
we need to load attachment.cgi as a GET before every POST, which
is a bit of a pain.
https://bugzilla.gnome.org/show_bug.cgi?id=700502
|
|
|
|
| |
Add $(VPATH), correct paths
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
See http://people.gnome.org/~walters/docs/build-api.txt
This way I can easily add it to the gnome-ostree manifest and have it
install into /usr, package people can do their thing, etc.
This dead simple non-autotools configure/Makefile system is adapted
from one I wrote for gtk-doc-stub.
Note I also added a --disable-documentation flag since gnome-ostree
doesn't have asciidoc.
|
|
|
|
|
|
|
| |
The code that applied plain diff tried to add a Unicode header
to uninterpreted patch data.
https://bugzilla.gnome.org/show_bug.cgi?id=687734
|
|
|
|
| |
Pointed out by Dan Winship
|
|
|
|
|
|
| |
Cookie jar got migrated from ~/.gnome2/epiphany to ~/.config/epiphany
https://bugzilla.gnome.org/show_bug.cgi?id=676241
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
| |
""" not """"
|
|
|
|
|
| |
Fix a double addition of the commit message, and add a comment
pointing out that the exact string is checked by git.gnome.org.
|
|
|
|
|
|
| |
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.
|
|
|
|
|
| |
If the editor uses the terminal, the missing _interactive=True will
cause things to hang.
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
| |
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
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
|
| |
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
|