Hi Vincent,

Thanks for feedback !

I'll try to answer some of your concerns,

Le 7 oct. 2012 14:34, "Vincent Massol" <[email protected]> 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
_______________________________________________
devs mailing list
[email protected]
http://lists.xwiki.org/mailman/listinfo/devs

Reply via email to