[Bug 426884] Review Request: eclipse-epic - Perl Eclipse plugin

bugzilla at redhat.com bugzilla at redhat.com
Tue Apr 29 02:56:39 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


https://bugzilla.redhat.com/show_bug.cgi?id=426884





------- Additional Comments From overholt at redhat.com  2008-04-28 22:56 EST -------
(In reply to comment #22)
> > - 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."
> :-)

That's fine by me :)

> > - are there any unit tests for EPIC which we can use to verify the JDOM API
> changes?
> 
> 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.

That works for me.

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

I think this might be:

http://www.redhat.com/archives/fedora-devel-list/2007-November/msg01948.html

which I'm probably seeing more so because I'm building on an F8 system.

-- 
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 mailing list