Skip to content
This repository has been archived by the owner on Jan 6, 2018. It is now read-only.

[3.1] OOZIE-114 put oozie-core out for maven repo publish #764

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

angeloh
Copy link

@angeloh angeloh commented Jun 2, 2011

OOZIE-114 put oozie-core out for maven repo publish

@@ -87,6 +87,11 @@
<source>${basedir}/../core/src/main/resources/oozie-default.xml</source>
<outputDirectory>/conf</outputDirectory>
</file>
<!-- Oozie core jar -->
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

There is a better way of doing this without using assembly

http://communitygrids.blogspot.com/2007/11/maven-making-war-and-jar-at-same-time.html

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I can see the jar is already there without this plugin.
$ ls -l webapp/target/oozie-webapp-3.1.0-SNAPSHOT/WEB-INF/lib/
-rw-r--r-- 1 angeloh users 1203296 Jun 6 16:15 oozie-core-3.1.0-SNAPSHOT.jar

Why using this plugin makes any difference?

The reason I am using this is to copy the oozie-core jar to distro target folder.

Copy link
Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

tucu,

Do you have any objections for this patch?

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants