Cyrus and Maven compatibility

[moosemorals]

Cyrus and Maven compatibility

[Moosemorals]

I'm happy for you to use any of the code (that's why its got a MIT licence!). I'm equally happy to take patches to fix bugs (and RFC incompatibilities are bugs).
I was aware that escapeString() was a very simple implementation - its why I use encodeString() to send to the server. Looking at the RFC, I think that Cyrus is wrong not to accept literal-c2s as the grammar says that "string" is quoted-string, literal-c2s or literal-s2c. On the other hand, since either string representation should work, I can't see an objection to using quoted string (other than the brain-deadness of my escapeString() implementation). I'm testing against dovecot, if that makes any difference....
Netbeans claims to support Maven, although I've never used it. There's already a Mavenised fork here on GitHub but I've never tried it, so don't let me stop you doing your own.
Using tabs instead of spaces may be a dealbreaker :-). (On the other hand, I can re-format anything you send back, so don't worry about it).
I'd suggest you open a bunch of issues for the separate things you've listed, and I'll close this one.
Thanks for the interest in my project!

[Emm]

On Fri, Feb 27, 2015 at 9:59 AM, Osric Wilkinson [email protected]
wrote:

I'm happy for you to use any of the code (that's why its got a MIT
licence!). I'm equally happy to take patches to fix bugs (and RFC
incompatibilities are bugs).
I was aware that escapeString() was a very simple implementation - its
why I use encodeString() to send to the server. Looking at the RFC, I
think that Cyrus is wrong not to accept literal-c2s as the grammar says
that "string" is quoted-string, literal-c2s or literal-s2c.

The RFC also states "This type of literal can only be sent from the server
to the client" when describing literal-s2c. So Dovecot is not compliant in
this regard.

On the other hand, since either string representation should work, I can't
see an objection to using quoted string (other than the brain-deadness of
my escapeString() implementation). I'm testing against dovecot, if that
makes any difference....
Netbeans claims to support Maven, although I've never used it. There's
already a Mavenised fork here on GitHub but I've never tried it, so don't
let me stop you doing your own.

This is actually an important point. Maven expects the code to be in
src/main/java (see the standard directory layout:
http://maven.apache.org/guides/introduction/introduction-to-the-standard-directory-layout.html).
This impacts the submission of patches. However, if moving the files is an
issue for you, it should be possible to patch the pom.xml file to point to
src instead of src/main/java.

Using tabs instead of spaces may be a dealbreaker :-). (On the other hand,
I can re-format anything you send back, so don't worry about it).

Ideally, we would like to see a mavenized ManageSieveJ released on
mvnrepository. This would mean we don't need to keep a fork-with-tabs in
our tree, we could just pull ManageSieveJ as a dependency.

I'd suggest you open a bunch of issues for the separate things you've
listed, and I'll close this one.
Thanks for the interest in my project!

You're welcome. Thanks for making the code available in the first place.

[Moosemorals]

Phew! That was a bunch of stuff. I've applied your patches (and thanks again for those), and I'm getting setup to add the project to Maven Central. Anything else on your wishlist?

[Emm]

That will be all. Thanks for the amazing (and fast) job.