[10:49:28 CST(-0600)] <drewwills> Are you there EricDalquist?

[10:50:01 CST(-0600)] <drewwills> I'm kicking the tires of pull #113 https://github.com/Jasig/uPortal/pull/113/files (bundled calendar portlet)

[10:50:43 CST(-0600)] <drewwills> The code changes (imho) look pretty elegant, and the whole thing works as expected

[10:51:14 CST(-0600)] <drewwills> the one issue I have is the number of extra build cycles in portlets, and the noise and time it adds to the build

[10:52:07 CST(-0600)] <drewwills> because both 'db-hibernate-portlets' and 'portlet-data-import' targets call $mvn package on up-portlets-overlay

[10:52:17 CST(-0600)] <EricDalquist> uhg

[10:52:25 CST(-0600)] <EricDalquist> yeah that would be good to minimize

[10:52:40 CST(-0600)] <drewwills> so an $ant initportal ends up packaging them 3 times

[10:52:43 CST(-0600)] <EricDalquist> also I've merged the work that james did into <plugin>

[10:52:43 CST(-0600)] <EricDalquist> <groupId>org.jasig.portal.maven</groupId>

[10:52:43 CST(-0600)] <EricDalquist> <artifactId>uportal-maven-plugin</artifactId>

[10:52:43 CST(-0600)] <EricDalquist> <version>1.0.0</version>

[10:52:43 CST(-0600)] <EricDalquist>

[10:52:43 CST(-0600)] <EricDalquist> <extensions>true</extensions>

[10:52:43 CST(-0600)] <EricDalquist> </plugin>

[10:52:47 CST(-0600)] <EricDalquist> yeah

[10:52:51 CST(-0600)] <EricDalquist> thats not great

[10:53:51 CST(-0600)] <drewwills> we could (1) skip rnning the package goal in these new tasks, or (2) somehow detect we're running within an initportal (or initdb) and skip it because we've just run it

[10:56:03 CST(-0600)] <drewwills> yep – we can use parameters in the <antcall> to shut off the package if we're coming from an aggregating task (like initportal): http://ant.apache.org/manual/index.html

[10:56:32 CST(-0600)] <EricDalquist> sounds good

[11:39:07 CST(-0600)] <drewwills> Got it worked out EricDalquist and it has the desired effect...

[11:39:14 CST(-0600)] <EricDalquist> cool

[11:39:21 CST(-0600)] <drewwills> but looking at the build logs, i find 1 thing surprising

[11:39:39 CST(-0600)] <drewwills> the portlet imports seem to be happening BEFORE the portal imports

[11:40:23 CST(-0600)] <drewwills> snippet: https://gist.github.com/drewwills/5058581

[11:40:40 CST(-0600)] <drewwills> portlet imports kick off after portlet-data-import:

[11:41:33 CST(-0600)] <drewwills> and after them, there's a up-shell: sectiuon, that starts the portal data

[11:41:51 CST(-0600)] <EricDalquist> yeah

[11:41:57 CST(-0600)] <EricDalquist> that is because the uportal data import is done via ant

[11:42:10 CST(-0600)] <EricDalquist> you'd have to switch the uportal data import over to use maven as well

[11:42:15 CST(-0600)] <EricDalquist> if you want it to run first

[11:42:17 CST(-0600)] <drewwills> complete log: https://gist.github.com/drewwills/5058590

[11:42:32 CST(-0600)] <drewwills> at this point i don't believe it matters

[11:43:23 CST(-0600)] <EricDalquist> not really

[11:43:35 CST(-0600)] <EricDalquist> and for 4.1 we can look at actually switching it over to use maven for data import

[11:43:42 CST(-0600)] <drewwills> sounds good

[11:44:14 CST(-0600)] <drewwills> this is really slick... i'm very pleased to see it emerge

[11:54:28 CST(-0600)] <drewwills> what about rel-4-0-patches... any reason to leave it out of there?

[11:54:47 CST(-0600)] <EricDalquist> for portlet import no

[11:54:50 CST(-0600)] <EricDalquist> but for uportal import

[11:54:54 CST(-0600)] <EricDalquist> that would be a big change

[11:55:02 CST(-0600)] <drewwills> yeah not that, sorry

[11:55:03 CST(-0600)] <EricDalquist> and could break existing build/deploy scripts that people have

[11:55:14 CST(-0600)] <drewwills> any reason not to bundle Calendar w/ 4-0-patches

[11:55:22 CST(-0600)] <EricDalquist> I don't think so

[11:55:27 CST(-0600)] <drewwills> me neither

[11:55:46 CST(-0600)] <EricDalquist> as you're merging

[11:55:54 CST(-0600)] <EricDalquist> can you switch the data plugin to: http://search.maven.org/#artifactdetails%7Corg.jasig.portal.maven%7Cuportal-maven-plugin%7C1.0.0%7Cmaven-plugin

[11:56:25 CST(-0600)] <drewwills> sure... semantics exactly the same?

[11:56:32 CST(-0600)] <EricDalquist> yes

[11:56:37 CST(-0600)] <drewwills> np

[11:56:39 CST(-0600)] <EricDalquist> I just merged the work into that

[11:56:57 CST(-0600)] <EricDalquist> it also a deploy-ear and deploy-ear-file goals

[11:57:06 CST(-0600)] <drewwills> yeah I caught it when you mentioned that... and I think it's a good idea

[11:57:32 CST(-0600)] <EricDalquist> we're working on getting more of our build into maven

[11:57:38 CST(-0600)] <EricDalquist> so I'll be working on that plugin off and on

[11:58:23 CST(-0600)] <drewwills> roger