Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Project migrated to Alfresco SDK 2.0 structure #6

Closed
wants to merge 3 commits into from

Conversation

douglascrp
Copy link

I don't know if it of your interest, but by using Alfresco SDK project you are able to execute your custom code inside an development Alfresco server.
Another advantage is to be able to package your code as an AMP, which is the official package format supported by Alfresco.
I did this port to be able to test this project, as I'm trying to implement the create document, as asked by @iwkse on the issue #3

If you accept my pull request, then I think you can move both alfresco and share projects to the project's root.
The alfresco project was created only to allow me to execute a full Alfresco server, since I need the repository part.

@douglascrp
Copy link
Author

@cetra3 I was looking at your github page, and i saw you have an alfresco gradle plugin project.
Based on that, I don't think you have the interest to port this project to Maven.
Please, let me know what you think.

@cetra3
Copy link
Owner

cetra3 commented Mar 23, 2015

@douglascrp Internally we use gradle for building our amps as it's much more discrete and we actually have our own amp creator plugin for gradle here:

https://bitbucket.org/parashift/alfresco-amp-plugin

This is what we use to build this project internally. I have not updated the build.gradle of this project to depend on this amp plugin as it's not available in maven central or jcenter, and we don't have a publicly accessible maven repo set up currently.

However: I do see the benefit of being able to build an amp file from maven, without gradle, as well. If you can recreate a pull request which just includes the pom.xml, and not change the folder layout considerably as in this pull request, that would be excellent!

@cetra3
Copy link
Owner

cetra3 commented Jun 29, 2015

@douglascrp OK for me to close this one?

@iwkse
Copy link

iwkse commented Jun 29, 2015

Hi Cetra,
I would like to ask you if you are somehow forced to use gradle for this project instead of the officially and diffused maven within Alfresco. I believe moving to maven would only let the project improves faster.
Regards

@cetra3
Copy link
Owner

cetra3 commented Jun 29, 2015

As I mentioned previously I'm not against maven. It's more just personal choice. Gradle is what we have been using internally to great effect.

I do see the benefit of having a Maven pom file, but this is something I don't have the bandwidth for doing myself at the moment.

@douglascrp
Copy link
Author

@cetra3 well, I already did that for you.
You can close it if you want, but it would be nice to have my pull request accepted.

@cetra3
Copy link
Owner

cetra3 commented Jun 30, 2015

There are a few extraneous files in your pull request, which I think are based upon the standard maven template which shouldn't really be included in this project. As I mentioned before, happy to accept the pull request, but I do need you to clean it up quite a bit before submitting.

@douglascrp
Copy link
Author

Ah, now I understand.
Yes, that's right... there are some "sample code" created by the use of
alfresco's archetype.
I'm going to perform a cleanup and I'm going to recreate the pull request.

Em seg, 29 de jun de 2015 às 22:14, cetra3 notifications@github.com
escreveu:

There are a few extraneous files in your pull request, which I think are
based upon the standard maven template which shouldn't really be included
in this project. As I mentioned before, happy to accept the pull request,
but I do need you to clean it up quite a bit before submitting.


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

@cetra3
Copy link
Owner

cetra3 commented Jul 15, 2015

I've made a commit to enable maven, so this pull request can close. Thanks for your help!

@cetra3 cetra3 closed this Jul 15, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants