BUG Ticket (closed)
BzrParser crashes parsing http://bzr.debian.org/pkg-bazaar/bzr-svn/experimental/
Source repository:
http://bzr.debian.org/pkg-bazaar/bzr-svn/experimental/
Stack trace:
{{{
undefined method `[]' for nil:NilClass
/slave/lib/scm/lib/scm/parsers/bzr_parser.rb:104:in `strip_trailing_asterisk'
/slave/lib/scm/lib/scm/parsers/bzr_parser.rb:99:in `parse_diffs'
/slave/lib/scm/lib/scm/parsers/bzr_parser.rb:89:in `parse_diffs'
/slave/lib/scm/lib/scm/parsers/bzr_parser.rb:71:in `internal_parse'
/slave/lib/scm/lib/scm/parsers/bzr_parser.rb:15:in `internal_parse'
/slave/lib/scm/lib/scm/parsers/parser.rb:12:in `parse'
/slave/lib/scm/lib/scm/adapters/bzr/commits.rb:51:in `each_commit'
/slave/lib/scm/lib/scm/adapters/bzr/commits.rb:84:in `open_log_file'
/slave/lib/scm/lib/scm/adapters/bzr/commits.rb:84:in `open_log_file'
/slave/lib/scm/lib/scm/adapters/bzr/commits.rb:50:in `each_commit'
}}}
on 02.12.09
reported by: robinluckey
on 02.13.09
by robinluckey
- **title** was changed to _BzrParser crashes parsing http://bzr.debian.org/pkg-bazaar/bzr-svn/experimental/_
- **description** was changed
- **id** was changed to _133_
One-line repro case:
{{{
bzr log -v -r 220.90.1 http://bzr.debian.org/pkg-bazaar/bzr-svn/experimental/ | ~/dev/scm/bin/ohlog --bzr
}}}
The problem appears to be the last line of the log for revision 200.90.1:
{{{
renamed:
subvertpy => subvertpy-20081002184530-hz9mrr3wqq4l8qdx-1
}}}
From the [online code browser](http://bazaar.launchpad.net/~jelmer/bzr-svn/experimental/revision/220.90.1), it appears that what's really happening is that directory `subvertpy/` is being renamed to `/`. Unfortunately, `/` appears in the log as an empty string.
Recognizing an empty string as the root directory should fix this parser crash.
However, this points to a much more serious bug around directory renaming: See [Ticket 10](/ohloh_scm/tickets/view/10).
on 02.13.09
by robinluckey
- **title** was changed to _BzrParser crashes parsing http://bzr.debian.org/pkg-bazaar/bzr-svn/experimental/_
- **description** was changed
- **status** was changed to _fixed_
- **id** was changed to _133_
Fixed in revision fe399d5.
Note however that this fix only prevents the crash when a directory is renamed to become the root directory.... it doesn't fix the related file rename issue in Ticket 10.
