Route53

[larskuhtz]

Hi Aristid,

Please consider merging the Route53 branch into the master branch. I would recommend that you have a quick lock at the Route53 code in order to ensure that it meats the quality bar and the coding style of the project.

Thanks,
Lars

[jberryman]

Is there a plan for a release on hackage that supports Route 53? Thanks!

[aristidb]

@jberryman There are a couple of things with this pull request that would need to be sorted out, so I don't think this can happen too soon.

The main problem right now, I think, is that this pull request introduces some additional dependencies which I don't want to put into the aws main package.

[larskuhtz]

Hi,

Unfortunately I currently have no free cycles to work on the Route53 code
and resolve the compatibility and dependency issues, but I am happy to
accept patches. Feel free to make any changes that you think are adequate
and to remove any dependencies that are an issue for the integration with
the AWS code. Probably, the current Route53 code won't work with recent
versions of the AWS package. So this would need fixing, too.

Alternatively, the Route53 code could be put into Hackage as a separate
package, but this should be subject to the code working with recent versions
of AWS.

Cheers,
Lars

From: Aristid Breitkreuz [email protected]
Reply-To: aristidb/aws
<reply+i-5588302-cb3966c455835425c316c6c2ccf6a27aa0ceefad-1369810@reply.gith
ub.com>
Date: Monday, January 7, 2013 11:17 AM
To: aristidb/aws [email protected]
Cc: Lars Kuhtz [email protected]
Subject: Re: [aws] Route53 (#25)

@jberryman https://github.com/jberryman There are a couple of things with
this pull request that would need to be sorted out, so I don't think this
can happen too soon.

The main problem right now, I think, is that this pull request introduces
some additional dependencies which I don't want to put into the aws main
package.

‹
Reply to this email directly or view it on GitHub
#25 (comment) .

[jberryman]

FWIW as a user I wouldn't mind seeing the package split up into many parts,
even to the level off having a separate package for stuff like signing API
calls. Thanks for the update and for your work on the package
On Jan 7, 2013 2:54 PM, "Lars Kuhtz" [email protected] wrote:

Hi,

Unfortunately I currently have no free cycles to work on the Route53 code
and resolve the compatibility and dependency issues, but I am happy to
accept patches. Feel free to make any changes that you think are adequate
and to remove any dependencies that are an issue for the integration with
the AWS code. Probably, the current Route53 code won't work with recent
versions of the AWS package. So this would need fixing, too.

Alternatively, the Route53 code could be put into Hackage as a separate
package, but this should be subject to the code working with recent
versions
of AWS.

Cheers,
Lars

From: Aristid Breitkreuz [email protected]
Reply-To: aristidb/aws

<reply+i-5588302-cb3966c455835425c316c6c2ccf6a27aa0ceefad-1369810@reply.gith
ub.com>
Date: Monday, January 7, 2013 11:17 AM
To: aristidb/aws [email protected]
Cc: Lars Kuhtz [email protected]
Subject: Re: [aws] Route53 (#25)

@jberryman https://github.com/jberryman There are a couple of things
with
this pull request that would need to be sorted out, so I don't think this
can happen too soon.

The main problem right now, I think, is that this pull request introduces
some additional dependencies which I don't want to put into the aws main
package.

‹
Reply to this email directly or view it on GitHub
#25 (comment) .

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/25#issuecomment-11968332.

[dterei]

Has there been any update on this?

[sseveran]

@dterei I have pushed a version that merges these changes and builds on the most recent version in our repo (https://github.com/alphaHeavy/aws/tree/Route53-test). I may work on breaking the xml-hamlet dependency.

[larskuhtz]

Hi Steve,

Thanks for updating the code. Would it be possible do the merge such
that the history of the Route53 code is preserved? That would make it
easier to track and review changes.

Thanks,
Lars

On 7/18/13 9:33 PM, Steve Severance wrote:

@dterei https://github.com/dterei I have pushed a version that
merges these changes and builds on the most recent version in our repo
(https://github.com/alphaHeavy/aws/tree/Route53-test). I may work on
breaking the xml-hamlet dependency.

—
Reply to this email directly or view it on GitHub
#25 (comment).

[dterei]

@aristidb Could you list what you'd like changed for this to be merged in please? Is it simply removing the xml-hamlet dependency or is there more?

[aristidb]

Well that's what I noticed first, I'd need to take a closer look to list
more things.
Am 19.07.2013 11:44 schrieb "David Terei" [email protected]:

@aristidb https://github.com/aristidb Could you list what you'd like
changed for this to be merged in please? Is it simply removing the
xml-hamlet dependency or is there more?

—
Reply to this email directly or view it on GitHubhttps://github.com//pull/25#issuecomment-21240755
.

[dterei]

For anyone looking for this code, we created an independent route53 package from this PR a while back and actively maintain it as we rely on it in production. It's essentially the code from this PR with some bug fixes and kept up-to-date for latest GHC / Hackage.

https://github.com/memcachier/aws-route53