Discussion:
[Be-devel] be import-xml fails
Matěj Cepl
2013-08-04 22:17:35 UTC
Permalink
I have exported a bug with be show --xml, changed a name of the bug (in
<summary> element; I don't know how to change name in other way), and
tried to reimport the bug via be import-xml. This is the sad result:

matej at wycliff: bugzilla-triage (updateRPCCalls *+%)$ be import-xml 9cd.xml
Traceback (most recent call last):
File "/home/matej/.local/bin/be", line 26, in <module>
sys.exit(libbe.ui.command_line.main())
File
"/home/matej/.local/lib/python2.7/site-packages/libbe/ui/command_line.py",
line 393, in main
ret = dispatch(ui, command, args)
File
"/home/matej/.local/lib/python2.7/site-packages/libbe/ui/command_line.py",
line 305, in dispatch
ret = ui.run(command, options, args)
File
"/home/matej/.local/lib/python2.7/site-packages/libbe/command/base.py",
line 590, in run
return command.run(options, args)
File
"/home/matej/.local/lib/python2.7/site-packages/libbe/command/base.py",
line 302, in run
self.status = self._run(**params)
File
"/home/matej/.local/lib/python2.7/site-packages/libbe/command/import_xml.py",
line 128, in _run
params, accept_changes, accept_extra_strings))
File
"/home/matej/.local/lib/python2.7/site-packages/libbe/command/import_xml.py",
line 255, in _merge_bugs
old = bugdir.bug_from_uuid(new.alt_id)
AttributeError: 'NoneType' object has no attribute 'bug_from_uuid'
matej at wycliff: bugzilla-triage (updateRPCCalls *+%)$

Known bug (I don't see anything familiar in be --server
http://cs.bugseverywhere.org/ list)?

Mat?j
--
http://www.ceplovi.cz/matej/, Jabber: mcepl at ceplovi.cz
GPG Finger: 89EF 4BC6 288A BF43 1BAB 25C3 E09F EF25 D964 84AC

My opinions may have changed, but not the fact that I am right.
--Ashleigh Brilliant
Matěj Cepl
2013-08-05 17:18:13 UTC
Permalink
Post by Matěj Cepl
I have exported a bug with be show --xml, changed a name of the bug (in
<summary> element; I don't know how to change name in other way), and
I have troubles to create proper merge request via menu (I have still
only one commit there), so let me ask you for this old-fashioned pull
request.

The following changes since commit dcbdaba0806b27fbe8f6a3e9b5ca7d18919be866:

Merge remote-tracking branch 'esr/master' (2013-08-02 10:51:52 -0700)

are available in the git repository at:


git://gitorious.org/~mcepl/be/mcepl-be.git import-xml-fail

for you to fetch changes up to fc1cd2eec9e36ce005f9240c24d17f12cdf3b55a:

Initialize bugdir variable in _merge_bugs need if empty.
(2013-08-05 17:21:44 +0200)

----------------------------------------------------------------
Mat?j Cepl (2):
A bit of PEP8 cleanup.
Initialize bugdir variable in _merge_bugs need if empty.

libbe/command/import_xml.py | 73 ++++++++++++++++++++++++++++-------------------
1 file changed, 43 insertions(+), 30 deletions(-)
W. Trevor King
2013-08-07 15:59:47 UTC
Permalink
Post by Matěj Cepl
Post by Matěj Cepl
I have exported a bug with be show --xml, changed a name of the
bug (in <summary> element; I don't know how to change name in
other way), and tried to reimport the bug via be import-xml. This
I have troubles to create proper merge request via menu (I have
still only one commit there), so let me ask you for this
old-fashioned pull request.
No problem. I'm going to merge locally anyway, so I prefer
old-fashioned pull requests to Gitorious' merge requests.
Post by Matěj Cepl
A bit of PEP8 cleanup.
This looks good to me :).
Post by Matěj Cepl
Initialize bugdir variable in _merge_bugs need if empty.
I'd rather fix this in line 110, with something like:

root_bugdir,root_bug,root_comment = (bugdirs.values()[0], None, None)

Thoughts?

Trevor
--
This email may be signed or encrypted with GnuPG (http://www.gnupg.org).
For more information, see http://en.wikipedia.org/wiki/Pretty_Good_Privacy
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 836 bytes
Desc: OpenPGP digital signature
URL: <http://void.printf.net/pipermail/be-devel/attachments/20130807/1f34b3f2/attachment.pgp>
Matěj Cepl
2013-09-02 12:30:45 UTC
Permalink
Post by W. Trevor King
root_bugdir,root_bug,root_comment = (bugdirs.values()[0], None, None)
Thoughts?
Did we merged this? I have created merge request for it
(http://gitorious.org/be/be/merge_requests/18 ... hopefully it contains
both commits I would like you to merge) just to be sure.

Best,

Mat?j
--
http://www.ceplovi.cz/matej/, Jabber: mcepl at ceplovi.cz
GPG Finger: 89EF 4BC6 288A BF43 1BAB 25C3 E09F EF25 D964 84AC

Why should I travel, when I'm already there?
-- Bostonian lady, when being asked why she never visited
other places than Boston

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 255 bytes
Desc: OpenPGP digital signature
URL: <http://void.printf.net/pipermail/be-devel/attachments/20130902/d9f426d1/attachment.pgp>
Loading...