7 Oct
2012
7 Oct
'12
2:46 p.m.
Some more feedback:
* You should put all your technical code in the MailArchiveCode space. I see you're
putting pages in existing spaces which is not that good: Scheduler, Extension, XApp.
* You should mark all the technical pages as hidden docs.
* You should have a package.xml file since it's generated automatically :)
* I see a java-libpst jar in some lib/ dir. This lib should be in maven if it's
required.
* MailArchive UI module is missing dependencies; it should depend on
xwiki-contrib-mailarchive-api for ex at least
Thanks
-Vincent
On Oct 7, 2012, at 2:34 PM, Vincent Massol <vincent(a)massol.net> wrote:
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.
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 ;-)
Thanks
-Vincent
PS: I'm still very eager to see it in action!