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

update xuggle repo #550

Merged
merged 2 commits into from
Aug 29, 2017
Merged

update xuggle repo #550

merged 2 commits into from
Aug 29, 2017

Conversation

junbaor
Copy link
Contributor

@junbaor junbaor commented Aug 28, 2017

No description provided.

@sarxos
Copy link
Owner

sarxos commented Aug 28, 2017

Hi @junbaor,

Thank you for this pull request, however I'm wondering if we have to relay on a 3rd-party private company (Liferay Inc.) to host these artifacts. In mvnrepository.com search page I found that Xuggler is available from the Open Source www.dcm4che.org repository at:

http://www.dcm4che.org/maven2/

Maybe we can use it instead? What do you think?

@junbaor
Copy link
Contributor Author

junbaor commented Aug 29, 2017

Hi @sarxos ,

Sorry , I just found out that

Note: this artifact it located at Dcm4Che repository (http://www.dcm4che.org/maven2/)

Thank you, www.dcm4che.org is indeed a better choice.

@sarxos
Copy link
Owner

sarxos commented Aug 29, 2017

Ok, I'm merging this pull request. Thank you @junbaor for your contribution!

@sarxos sarxos merged commit 6a5b30a into sarxos:master Aug 29, 2017
@junbaor junbaor deleted the patch-1 branch August 29, 2017 07:33
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.

2 participants