public inbox for mauve-discuss@sourceware.org
 help / color / mirror / Atom feed
From: Mark Wielaard <mark@klomp.org>
To: Fabien DUMINY <fabien.duminy@webmails.com>
Cc: mauve-discuss@sourceware.org
Subject: Re: version of java for the sources ?
Date: Thu, 13 Nov 2008 11:20:00 -0000	[thread overview]
Message-ID: <1226575194.4563.4.camel@dijkstra.wildebeest.org> (raw)
In-Reply-To: <491BC174.4000108@webmails.com>

Hi Fabien,

On Thu, 2008-11-13 at 06:56 +0100, Fabien DUMINY wrote:
> Can I use Java 5 features in mauve sources (the core part, not the 
> testlet part) ?

We do normally try to not be too fancy in mauve, just because there are
still some older vms and compilers out there. So if you can avoid using
these newer language constructs that would be appreciated.

> Here are the changes I am currently finalizing :
> - an XML report reader/writer
> - a comparator of 2 XML reports (to show regressions/progressions in 
> both html or plain text)
> - some non-JNode-specific minor changes that were needed to make mauve 
> usable in JNode
> - removing gnu.testlet.runner.XMLGenerator : that class is empty and I 
> am not modifying/using it

Very nice!

If it is hard to avoid the jdk5 langauge features and you can contain
them to just the XML generation part then we could probably just make
them optional in case someone needs a "plain old mauve without bells".

> - add nanoxml lite (licence zlib/libpng) by copying the 2 files 
> (XMLElement, XMLParseException) from icedtea

That should be fine. But please do make sure to prominently document
where they came from and how to update them to upstream versions in the
future if necessary.

Cheers,

Mark

  reply	other threads:[~2008-11-13 11:20 UTC|newest]

Thread overview: 3+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2008-11-13  5:56 Fabien DUMINY
2008-11-13 11:20 ` Mark Wielaard [this message]
2008-11-13 11:44 ` Christian Thalinger

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1226575194.4563.4.camel@dijkstra.wildebeest.org \
    --to=mark@klomp.org \
    --cc=fabien.duminy@webmails.com \
    --cc=mauve-discuss@sourceware.org \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).