7 Oct
2012
7 Oct
'12
5:55 p.m.
Hi Vincent,
Thanks for feedback !
I'll try to answer some of your concerns,
Le 7 oct. 2012 14:34, "Vincent Massol" <vincent(a)massol.net> a écrit :
Hi Jeremie,
I've tried using version 0.1 this week end and I've found that it doesn't
work and is missing things. For example, it requires the tab macro which isn't
specified as a dependency. And some pages have velocity macro failures.
Yep I'll add that dependency, I've created XMAILARCH-23 to track that.
Many pages have velocity macro failures, some of them are fixed in
coming 0.2 (I hope) .
I've then tried to build the mailarchive app from
git in the hope that some more recent commits fixed the issues but I couldn't built
it. The build fails with:
"Results :
Failed tests:
decodeMailContentForHtmlAndNoBodyAndCut(org.xwiki.contrib.mailarchive.internal.MailUtilsTest):
To be implemented
Tests in error:
extractTypesWithLimitValues(org.xwiki.contrib.mailarchive.internal.DefaultMailArchiveTest):
Failed to lookup component [org.xwiki.contrib.mailarchive.internal.DefaultMailArchive] for
role [org.xwiki.contrib.mailarchive.IMailArchive] and hint [default]
extractTypesForNominalCases_OtherType(org.xwiki.contrib.mailarchive.internal.DefaultMailArchiveTest):
Failed to lookup component [org.xwiki.contrib.mailarchive.internal.DefaultMailArchive] for
role [org.xwiki.contrib.mailarchive.IMailArchive] and hint [default]
extractTypesForMultiplePatternsAndTypes(org.xwiki.contrib.mailarchive.internal.DefaultMailArchiveTest):
Failed to lookup component [org.xwiki.contrib.mailarchive.internal.DefaultMailArchive] for
role [org.xwiki.contrib.mailarchive.IMailArchive] and hint [default]
Tests run: 10, Failures: 1, Errors: 3, Skipped: 0
"
I've also noticed that on master the core dep is 3.5.1 which is pretty old. I'd
like to update this to 4.3-SNAPSHOT and when you perform a release you resolve the
snapshot to whatever version is the latest released (ATM that would be 4.2 and later on
that would be 4.3-milestone-1 for ex).
If you still need to depend on 3.5.1, it should be done on a branch of the mail archive
app IMO.
It would be nice for ex to be able to use the Application panel extension point.
WDYT?
I'm now going to try building with -DskipTests and pray it's going to run ;-)
Currently the state of code under git is not good ... I fixed a number
of things but it got out of my mind to commit them. I'll fix that
right away.
Thanks
-Vincent
PS: I'm still very eager to see it in action!
That's a good thing :) It still is really in alpha stage, that's a
pity because the old version (groovysh one) is still up and running
quite well, I really want to deliver a better and nearly usable 0.2 :)
Br,
Jeremie