[Bug 426884] Review Request: eclipse-epic - Perl Eclipse plugin
bugzilla at redhat.com
bugzilla at redhat.com
Mon Apr 28 21:19:02 UTC 2008
Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.
Summary: Review Request: eclipse-epic - Perl Eclipse plugin
------- Additional Comments From fedora at matbooth.co.uk 2008-04-28 17:19 EST -------
Created an attachment (id=304046)
Variables View Screenshot
Sorry guys, yes PadWalker should be required. I must have already had it
installed. It's a bit concerning you can't see any variables even after
installing it. I've attached an example screenshot of what you should be
> - there are lots of antlr warnings - do you think these are okay?
To be honest, I don't really know what half of those antlr warnings even mean
since grammar parsing is not something I've ever done myself. Everything seems
to work ok though, so my philosophy on this has been "warnings are not errors."
I will post links to binary rpms in a minute for you Jack, maybe you can have a
quick check to see if there is anything obviously wrong with the syntax
highlighting, auto-completion and what have you.
> - are there any unit tests for EPIC which we can use to verify the JDOM API
I don't think so, upstream admits his test suite is "very incomplete." However,
if you add paths to your project's Perl Include Path property sheet, it
correctly saves an xml .includepath settings file in your project directory and
is able to find external Perl modules using it. This is only place where the
code I've patched is used.
> - any idea what's going on with the debuginfo extraction? I get lots of
> the following. I'm not super-concerned, since this is gcj debuginfo
> and the chances of anyone other than gcj developers being able to use
> it successfully is low :)
This one I'm not sure about. I just added the gcj lines that the guidelines
told me to add in, so I figured it was normal. Would it be worth asking someone
who knows a bit more about it to see if it's a problem?
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug, or are watching someone who is.
More information about the package-review