uPortal IRC Logs-2011-10-24

Owned by Apereo Infrastructure
Last updated: Oct 24, 2011Version comment
14 min read

[09:42:56 CDT(-0500)] <Arvids> What do you think about moving uPortal CAS integration into seperate maven module?

[09:43:17 CDT(-0500)] <Arvids> i.e. the same as uportal-search-api

[09:46:14 CDT(-0500)] <Arvids> It would allow to create CAS more detachable (along with all dependencies)

[09:54:33 CDT(-0500)] <EricDalquist> what parts are included in the cas integration?

[09:57:40 CDT(-0500)] <Arvids> Everything from org.jasig.portal.security.provider.cas package

[09:59:17 CDT(-0500)] <EricDalquist> right but all of that is 100% uPortal specific

[09:59:47 CDT(-0500)] <EricDalquist> and since it depends on ISecurityContext unless we split out a uPortal API module we have a circular depenency

[09:59:59 CDT(-0500)] <Arvids> ahh.. didn't think of that

[10:00:07 CDT(-0500)] <EricDalquist> as the cas-security module would reference uportal-war which would reference cas-security

[10:00:08 CDT(-0500)] <EricDalquist> (tongue)

[10:00:18 CDT(-0500)] <EricDalquist> I'd love to split up uPortal into lots of little modules

[10:00:24 CDT(-0500)] <EricDalquist> but it is some serious work

[10:00:56 CDT(-0500)] <EricDalquist> and would have to be done by starting with interfaces/APIs

[10:00:56 CDT(-0500)] <Arvids> it would require a very good and solid architecture

[10:01:00 CDT(-0500)] <EricDalquist> yes

[10:01:10 CDT(-0500)] <EricDalquist> that is actually why I'd like to do it

[10:01:32 CDT(-0500)] <EricDalquist> the structure of the project would help enforce better architecture

[10:01:49 CDT(-0500)] <Arvids> Spring security might solve a lot of issues in security section

[10:01:54 CDT(-0500)] <EricDalquist> yes

[10:01:58 CDT(-0500)] <EricDalquist> that is a big TODO on the list

[10:02:08 CDT(-0500)] <EricDalquist> completely replace the AuthN/AuthZ checks with spring security

[10:02:22 CDT(-0500)] <Arvids> even move cas integration into separate module

[10:02:57 CDT(-0500)] <EricDalquist> well spring security has its own cas integration

[10:03:00 CDT(-0500)] <EricDalquist> I believe

[10:03:22 CDT(-0500)] <Arvids> we're using ForgeRock's Access Manager

[10:03:34 CDT(-0500)] <Arvids> and i know it has Spring Security integration

[10:03:53 CDT(-0500)] <Arvids> currently we're using a custom login module

[11:49:01 CDT(-0500)] <EricDalquist> question on portal eventing

[11:49:22 CDT(-0500)] <EricDalquist> should things the portal render or portlet execution events still fire if an exception is thrown?

[11:49:37 CDT(-0500)] <EricDalquist> or should I be creating error specific events

[11:51:14 CDT(-0500)] <athena> huh.

[11:51:48 CDT(-0500)] <athena> i'm really not sure

[11:52:16 CDT(-0500)] <EricDalquist> I'm thinking that I might just have an Exception as an optional argument when firing the event

[11:59:14 CDT(-0500)] <athena> that makes sense

[12:01:05 CDT(-0500)] <EricDalquist> I'm happy with the json approach

[12:01:18 CDT(-0500)] <EricDalquist> makes adding new event types or new fields MUCH easier

[12:01:43 CDT(-0500)] <athena> nice!

[12:49:28 CDT(-0500)] <athena> argh

[12:49:35 CDT(-0500)] <EricDalquist> ?

[12:49:37 CDT(-0500)] <athena> git.

[12:49:42 CDT(-0500)] <EricDalquist> uhoh

[12:49:43 CDT(-0500)] <EricDalquist> whats up?

[12:50:01 CDT(-0500)] <athena> really no idea how to actually get these changes back to uportal

[12:50:11 CDT(-0500)] <EricDalquist> so did you do a personal fork?

[12:50:15 CDT(-0500)] <athena> yeah

[12:50:16 CDT(-0500)] <EricDalquist> or did you just clone the uPortal repo

[12:50:17 CDT(-0500)] <EricDalquist> ok

[12:50:23 CDT(-0500)] <EricDalquist> so that isn't really needed for people with commit access

[12:50:25 CDT(-0500)] <athena> should i just give up and go back and clone the uportal repo instead?

[12:50:32 CDT(-0500)] <athena> if that makes things easier i'm happy to just go do that

[12:50:32 CDT(-0500)] <EricDalquist> but there are two ways to get your changes back to uPortal

[12:50:42 CDT(-0500)] <EricDalquist> so what have you done so far

[12:50:49 CDT(-0500)] <EricDalquist> did you create a branch in your fork?

[12:50:52 CDT(-0500)] <athena> yes

[12:50:59 CDT(-0500)] <EricDalquist> and you have your changes complete?

[12:51:02 CDT(-0500)] <athena> and friday we added the jasig uportal trunk as an upstream remote

[12:51:08 CDT(-0500)] <EricDalquist> ah right

[12:51:08 CDT(-0500)] <athena> yes, and committed them to my little local branch

[12:51:11 CDT(-0500)] <EricDalquist> ok

[12:51:20 CDT(-0500)] <EricDalquist> so do a "git push" on your branch

[12:51:28 CDT(-0500)] <EricDalquist> which will push your changes into your fork

[12:51:28 CDT(-0500)] <athena> ah, ok

[12:51:33 CDT(-0500)] <EricDalquist> on github

[12:51:42 CDT(-0500)] <EricDalquist> then you can go to github and do a pull request

[12:51:49 CDT(-0500)] <EricDalquist> and then accept it yourself

[12:52:14 CDT(-0500)] <EricDalquist> you can also get the changes directly back into uPortal, its just a little more complex

[12:52:26 CDT(-0500)] <EricDalquist> and I'd just clone the uPortal repo

[12:52:33 CDT(-0500)] <EricDalquist> and work there

[12:52:36 CDT(-0500)] <EricDalquist> less complicated

[12:52:36 CDT(-0500)] <athena> yeah, i'm just going to go do that

[12:52:49 CDT(-0500)] <athena> do we have a writeup of the expected workflow for working directly w/ the uportal repo?

[12:52:52 CDT(-0500)] <EricDalquist> and if you ever want someone to review your work you can just point them to the github branch compare page

[12:52:57 CDT(-0500)] <EricDalquist> not yet

[12:53:02 CDT(-0500)] <EricDalquist> I'll be working on that tomorrow

[12:53:07 CDT(-0500)] <athena> hm, ok

[12:53:16 CDT(-0500)] <EricDalquist> but it should be something like:

[12:53:59 CDT(-0500)] <EricDalquist> clone

[12:53:59 CDT(-0500)] <EricDalquist> pull, branch UP-xxxx, do work, push

[12:54:16 CDT(-0500)] <EricDalquist> then "checkout master, merge up-xxxx, push, delete branch" when done

[12:54:40 CDT(-0500)] <athena> ok

[12:54:52 CDT(-0500)] <athena> i'll try that out today

[12:55:00 CDT(-0500)] <EricDalquist> I'll see if I can get a skeleton page up soon

[13:01:29 CDT(-0500)] <EricDalquist> ok, lunch time, I should have basic directions posted in a few hours

[13:01:37 CDT(-0500)] <athena> awesome, thanks

[14:37:33 CDT(-0500)] <EricDalquist> jen looks like there is some lingering dependency on resource server utils 22-snapshot

[14:42:13 CDT(-0500)] <athena> lingering dependency?

[14:43:50 CDT(-0500)] <EricDalquist> er I guess I miss read that

[14:43:54 CDT(-0500)] <EricDalquist> unfulfilled dependency

[14:44:06 CDT(-0500)] <EricDalquist> org.jasig.resourceserver:resource-server-utils:jar:1.0.22-SNAPSHOT

[14:44:20 CDT(-0500)] <EricDalquist> weird

[14:44:25 CDT(-0500)] <EricDalquist> it looks like it downloaded it earlier

[14:44:35 CDT(-0500)] <athena> yeah it looks like it's out on sonatype

[14:44:41 CDT(-0500)] <athena> https://oss.sonatype.org/content/repositories/snapshots/org/jasig/resourceserver/resource-server-utils/1.0.22-SNAPSHOT/

[14:44:44 CDT(-0500)] <EricDalquist> I wonder if it some ant/maven issue :/

[14:44:49 CDT(-0500)] <athena> dunno

[14:44:49 CDT(-0500)] <EricDalquist> I'm going to try re-runnign the build

[14:44:57 CDT(-0500)] <athena> every once in a while i've had maven get into some weird state with a snapshot

[14:45:05 CDT(-0500)] <EricDalquist> also fluid provided some very good git info

[14:45:05 CDT(-0500)] <athena> and had to delete that directory from my repository force it to re-download

[14:45:08 CDT(-0500)] <athena> oh terrific

[14:45:25 CDT(-0500)] <athena> they're doing the push-approve style thing, right?

[14:45:31 CDT(-0500)] <EricDalquist> yes

[14:45:40 CDT(-0500)] <EricDalquist> but they also document how to have one local repository

[14:45:51 CDT(-0500)] <EricDalquist> which is linked both to your fork and to the central repo

[14:45:53 CDT(-0500)] <EricDalquist> so you can push to both

[14:45:58 CDT(-0500)] <athena> oh nice

[14:46:00 CDT(-0500)] <EricDalquist> allowing you to do pull requests if you want

[14:46:04 CDT(-0500)] <EricDalquist> or just push directly

[14:46:17 CDT(-0500)] <EricDalquist> so I think that is what I'll document as well

[14:47:43 CDT(-0500)] <athena> sounds good

[14:53:14 CDT(-0500)] <athena> EricDalquist: looks like the problem w/ the resource server is that the current codebase is pointing to a snapshot of the jasig-parent project

[14:53:22 CDT(-0500)] <athena> and that's what it actually cant' find

[14:53:28 CDT(-0500)] <EricDalquist> ah

[14:53:31 CDT(-0500)] <EricDalquist> weird

[14:53:47 CDT(-0500)] <athena> looks like that version is tagged now though

[14:54:00 CDT(-0500)] <athena> so i can just update the resource server to do that and run a mvn deploy

[14:54:26 CDT(-0500)] <EricDalquist> sounds good

[15:10:40 CDT(-0500)] <athena> do these commits look ok?

[15:10:43 CDT(-0500)] <athena> put a few in

[15:11:39 CDT(-0500)] <EricDalquist> yup, though you didn't need to do the test as its own commit

[15:11:44 CDT(-0500)] <EricDalquist> one other VERY neat feature of git

[15:11:52 CDT(-0500)] <EricDalquist> is you can ammend your last commit locally

[15:11:57 CDT(-0500)] <athena> ah, good point

[15:12:02 CDT(-0500)] <athena> could have gone back and fixed that

[15:12:05 CDT(-0500)] <EricDalquist> so if you commit something, then realize you messed up and forgot one bit

[15:12:10 CDT(-0500)] <athena> yeah

[15:12:15 CDT(-0500)] <athena> i'll have to try that next time

[15:12:16 CDT(-0500)] <EricDalquist> you can do another commit with —amend (I believe)

[15:12:19 CDT(-0500)] <athena> yeah

[15:12:24 CDT(-0500)] <athena> do remember seeing something about that now

[15:12:26 CDT(-0500)] <EricDalquist> as long as you havn't pushed yet

[15:12:32 CDT(-0500)] <EricDalquist> you can amend after a push

[15:12:36 CDT(-0500)] <EricDalquist> but it can get bad

[15:12:43 CDT(-0500)] <athena> bad sounds no good (smile)

[15:12:54 CDT(-0500)] <EricDalquist> since anyone that has pulled down your pre-amend commit has do do some work to re-wind and replay

[15:13:22 CDT(-0500)] <athena> gotcha

[15:13:26 CDT(-0500)] <athena> yeah that sounds bad

[15:13:37 CDT(-0500)] <EricDalquist> have you discovered the branch list and branch compare view yet?

[15:13:43 CDT(-0500)] <EricDalquist> https://github.com/Jasig/uPortal/branches

[15:13:44 CDT(-0500)] <athena> yeah

[15:14:44 CDT(-0500)] <athena> neat stuff

[15:16:07 CDT(-0500)] <athena> think this will be a really good change for us on the whole

[15:16:31 CDT(-0500)] <EricDalquist> yes, it will just take a little bit of learning

[15:16:34 CDT(-0500)] <athena> yeah

[15:16:36 CDT(-0500)] <athena> definitely that (smile)

[15:16:42 CDT(-0500)] <athena> least i can commit some stuff now

[15:16:45 CDT(-0500)] <athena> progress.

[15:16:54 CDT(-0500)] <athena> might be worth talking about when/how we migrate the portlets

[15:17:06 CDT(-0500)] <athena> might be some advantage to doing the officially sponsored ones all together

[15:17:11 CDT(-0500)] <athena> worry it'll get a bit confusing otherwise

[15:17:30 CDT(-0500)] <EricDalquist> yeah

[15:17:36 CDT(-0500)] <holdorph> maintainers should each have their own choice

[15:17:46 CDT(-0500)] <EricDalquist> though really it is more up to the lead for each portlet project

[15:17:49 CDT(-0500)] <holdorph> i don't think they all should get lumped together, unless they all agreeed

[15:17:58 CDT(-0500)] <holdorph> exactly

[15:18:02 CDT(-0500)] <athena> yes

[15:18:26 CDT(-0500)] <athena> though in reality the portlet leads are a very, very small group of people, most of whom are uportal committers

[15:19:03 CDT(-0500)] <holdorph> small, maybe, but doesn't mean they should be without a choice

[15:19:12 CDT(-0500)] <athena> didn't say they shouldn't have a choice

[15:19:27 CDT(-0500)] <athena> said we should talk about it and maybe consider moving them at the same time, provided that people agree that seems sensible

[15:19:36 CDT(-0500)] <EricDalquist> yup (smile)

[15:19:56 CDT(-0500)] <EricDalquist> I do agree that it would be good to try and avoid the potential confusion of having portlet source in various places

[15:20:22 CDT(-0500)] <athena> not quite as bad if we set up svn mirrors for them all, i suppose, since then the "old" url could continue to work for a while

[15:20:34 CDT(-0500)] <athena> but it may still get confusing

[15:20:38 CDT(-0500)] <EricDalquist> yeah

[15:20:47 CDT(-0500)] <EricDalquist> though I'd REALLY like to avoid svn mirroring becoming the norm

[15:20:53 CDT(-0500)] <EricDalquist> while it isn't terribly complicated

[15:21:00 CDT(-0500)] <EricDalquist> it is a lot of work to setup and maintain

[15:21:06 CDT(-0500)] <athena> gotcha

[15:21:21 CDT(-0500)] <EricDalquist> I still have to finish figuring out the bamboo config to get automated syncing working

[15:21:49 CDT(-0500)] <athena> so do we have some new interesting options for maintaining local versions now? i assume there's probably a way to have a university-specific git project that's based off of uportal's in some useful way?

[15:22:13 CDT(-0500)] <EricDalquist> and a better solution for smaller portlet projects is to make sure we provide the ability to do useful overlays

[15:22:15 CDT(-0500)] <EricDalquist> yes

[15:22:43 CDT(-0500)] <EricDalquist> anyone could create a clone of the uPortal git repository onto some campus specific git server

[15:22:53 CDT(-0500)] <holdorph> foggy memory, here... am I remembering right, that erik at UC irvine moved on, and is no longer doing their portal work?

[15:22:54 CDT(-0500)] <EricDalquist> and then maintain it just like you or I would maintain a github work

[15:22:58 CDT(-0500)] <athena> was hoping that was the case

[15:23:00 CDT(-0500)] <EricDalquist> that is correct

[15:23:03 CDT(-0500)] <athena> yeah (sad)

[15:23:15 CDT(-0500)] <holdorph> so this page is out of date then? https://wiki.jasig.org/display/PLT/Announcements+Portlet

[15:23:22 CDT(-0500)] <EricDalquist> yes

[15:23:25 CDT(-0500)] <athena> yeah

[15:23:32 CDT(-0500)] <athena> i've actually made a few commits to that portlet lately

[15:23:45 CDT(-0500)] <athena> need to send out an email asking for some feedback and organizing a new release of it

[15:23:57 CDT(-0500)] <athena> make sure it gets into maven, etc.

[15:24:04 CDT(-0500)] <EricDalquist> and even better athena is that they could also have a github clone

[15:24:14 CDT(-0500)] <EricDalquist> and then push local fixes to their github clone

[15:24:20 CDT(-0500)] <EricDalquist> and pull req them right back into uPortal

[15:24:20 CDT(-0500)] <athena> nice

[15:24:34 CDT(-0500)] <athena> yeah i think we'll have some good options in the future

[15:24:44 CDT(-0500)] <EricDalquist> yup

[15:24:59 CDT(-0500)] <EricDalquist> though I still want to better document some standards for portlet projects to make overlays easier

[15:25:03 CDT(-0500)] <EricDalquist> like spring app context overrides

[15:25:05 CDT(-0500)] <athena> i'd love to see if we can turn JSPC off in the announcements portlet by default

[15:25:09 CDT(-0500)] <EricDalquist> generated a -classes artifact

[15:25:11 CDT(-0500)] <athena> yeah

[15:25:18 CDT(-0500)] <athena> JSPC makes it impossible to overlay JSP files (sad)

[15:25:30 CDT(-0500)] <athena> or really hard, anyway

[15:25:42 CDT(-0500)] <EricDalquist> and to use them on any container other than the exact version it was JSPCd for

[15:25:49 CDT(-0500)] <EricDalquist> JSPC should be part of the local overlay step

[15:25:53 CDT(-0500)] <athena> yeah

[15:25:53 CDT(-0500)] <EricDalquist> never part of a base portlet

[15:26:08 CDT(-0500)] <athena> sounds good to me

[15:28:39 CDT(-0500)] <athena> so i checked out uportal in the W3C validator

[15:28:42 CDT(-0500)] <athena> only a few bugs

[15:28:53 CDT(-0500)] <EricDalquist> nice

[15:28:56 CDT(-0500)] <athena> yeah

[15:29:05 CDT(-0500)] <athena> think we could probably at least get the base theme itself validating

[15:29:20 CDT(-0500)] <EricDalquist> that would be good

[15:29:23 CDT(-0500)] <athena> we may have a couple invalid spots because we're doing elision w/ fluid

[15:29:30 CDT(-0500)] <EricDalquist> and it would be nice to get bundled portlets in better shape

[15:29:32 CDT(-0500)] <athena> yeah

[15:29:38 CDT(-0500)] <athena> need to add i18n support too

[15:29:47 CDT(-0500)] <EricDalquist> can we use namespaces in html5?

[15:29:54 CDT(-0500)] <EricDalquist> and would that solve the fluid elision issues?

[15:30:04 CDT(-0500)] <athena> dunno if that would help

[15:30:21 CDT(-0500)] <athena> the issue with elision is we have stuff like <ul><div><li></li></div></ul>

[15:30:24 CDT(-0500)] <athena> which isnt' really valid

[15:30:33 CDT(-0500)] <athena> but actually the div gets stripped out of the markup - it's just sort of a temporary container

[15:30:47 CDT(-0500)] <EricDalquist> ah ok

[15:30:54 CDT(-0500)] <athena> will have to ask them about that

[15:31:01 CDT(-0500)] <athena> jQuery mobile has a bit of that too, i think

[15:31:06 CDT(-0500)] <EricDalquist> yeah

[15:31:18 CDT(-0500)] <EricDalquist> it would be nice if it would validate the post JS version of the DOM

[15:31:32 CDT(-0500)] <athena> yeah

[15:31:39 CDT(-0500)] <athena> no idea how to do that (smile)

[15:34:57 CDT(-0500)] <EricDalquist> athena: https://wiki.jasig.org/display/UPC/Git+Tips+and+Tricks

[15:35:00 CDT(-0500)] <EricDalquist> still in progress

[15:35:06 CDT(-0500)] <athena> awesome, thanks!

[15:35:13 CDT(-0500)] <EricDalquist> and I think I have the eventing code all set for an initial merge

[15:35:25 CDT(-0500)] <EricDalquist> I do need to get the aggregation done before I merge though

[15:35:32 CDT(-0500)] <EricDalquist> the aggregation framework I mean

[15:35:35 CDT(-0500)] <EricDalquist> so that purging is in place

[15:35:56 CDT(-0500)] <athena> very nice

[15:45:54 CDT(-0500)] <athena> EricDalquist: is the current trunk building ok for you?

[15:46:03 CDT(-0500)] <athena> all of the sudden i'm getting test failures

[15:46:07 CDT(-0500)] <EricDalquist> um … let me check

[15:46:38 CDT(-0500)] <EricDalquist> and this is why I like git

[15:46:44 CDT(-0500)] <athena> weird stuff, too,.

[15:46:46 CDT(-0500)] <EricDalquist> "git add .; git stash; git checkout master"

[15:46:46 CDT(-0500)] <athena> <error message="Invalid index" type="sun.misc.InvalidJarIndexException">sun.misc.InvalidJarIndexException: Invalid index

[15:46:46 CDT(-0500)] <athena> at sun.misc.URLClassPath$JarLoader.getResource(URLClassPath.java:858)

[15:46:55 CDT(-0500)] <EricDalquist> and now I can help Jen test

[15:47:01 CDT(-0500)] <athena> hurray! (smile)

[15:47:02 CDT(-0500)] <athena> lol

[15:48:38 CDT(-0500)] <athena> really not clear on what that error message is

[15:50:00 CDT(-0500)] <EricDalquist> ok just verified that my current repo builds, pulling down changes ...

[15:53:29 CDT(-0500)] <EricDalquist> ok build fails now that I've pulled in your 3 changes (wink)

[15:53:33 CDT(-0500)] <EricDalquist> testImportEntityTypes(org.jasig.portal.io.ImportTest): The Cernunnos Runtime encountered an error:

[15:53:33 CDT(-0500)] <EricDalquist> testFailure(org.jasig.portal.tools.checks.ClassPresenceCheckTest): Invalid index

[15:54:04 CDT(-0500)] <EricDalquist> huh wow yeah: Caused by: sun.misc.InvalidJarIndexException: Invalid index

[15:54:08 CDT(-0500)] <athena> yeah, that's what i'm seeing too

[15:54:16 CDT(-0500)] <athena> didn't change anything in those classes

[15:54:23 CDT(-0500)] <athena> and a clean build ran fine before i committed

[15:54:26 CDT(-0500)] <athena> so really confused

[15:54:54 CDT(-0500)] <athena> those tests don't even seem related to what i committed

[15:57:01 CDT(-0500)] <EricDalquist> wtf … this is VERY weird

[15:57:03 CDT(-0500)] <athena> yeah.

[15:57:27 CDT(-0500)] <athena> bad jar from maven or something?

[15:58:01 CDT(-0500)] <EricDalquist> but for bot of us to have it and the build server?

[15:58:10 CDT(-0500)] <athena> no idea.

[15:58:55 CDT(-0500)] <athena> and cant' even really tell from that stack trace what it might be

[15:58:56 CDT(-0500)] <EricDalquist> since no dependencies changed with these commits

[15:59:10 CDT(-0500)] <EricDalquist> and my build worked before I pulled the 3 of them on

[15:59:11 CDT(-0500)] <EricDalquist> in

[15:59:19 CDT(-0500)] <EricDalquist> so I'm going to revert all 3 and apply them one at a time

[15:59:23 CDT(-0500)] <EricDalquist> git reset --hard HEAD~3

[15:59:50 CDT(-0500)] <EricDalquist> that rolls back your current copy by 3 commits

[16:03:24 CDT(-0500)] <EricDalquist> yeah it is the fluid commit

[16:03:31 CDT(-0500)] <EricDalquist> and I realized it does touch the resource server

[16:05:42 CDT(-0500)] <athena> yeah, pulling in a new version of the resource server

[16:05:54 CDT(-0500)] <athena> was just building it locally to see if it's the version-in-the-sky that's causing problems

[16:06:41 CDT(-0500)] <EricDalquist> ok

[16:06:44 CDT(-0500)] <EricDalquist> I think it is

[16:06:54 CDT(-0500)] <athena> yeah, wonder if it got corrupted or something?

[16:06:55 CDT(-0500)] <EricDalquist> I just nuked 1.0.22* from my local repo

[16:06:58 CDT(-0500)] <EricDalquist> and rebuilt it

[16:07:22 CDT(-0500)] <EricDalquist> could be … could be some problem with the stupid jarjar stuff we have to do (I really HATE yui's repackaging)

[16:09:55 CDT(-0500)] <athena> did rebuilding help? doesn't seem to have done it for me

[16:10:18 CDT(-0500)] <EricDalquist> I didn't try a local rebuild, I don't have resource server checked out on my laptop (I'm at home today)

[16:11:24 CDT(-0500)] <athena> gotcha

[16:11:31 CDT(-0500)] <athena> not helping so much here

[16:11:47 CDT(-0500)] <athena> does reverting back to version 21 let the tests run ok?

[16:12:02 CDT(-0500)] <EricDalquist> yup

[16:12:08 CDT(-0500)] <athena> huh, yeah, that is weird

[16:12:11 CDT(-0500)] <EricDalquist> if I revert that fluid commit it works

[16:12:25 CDT(-0500)] <athena> yeah, wanted to make sure it was just the resource server version, not something else in that commit

[16:12:56 CDT(-0500)] <EricDalquist> oh, I didn't check that but I'd be VERY suprised if that wasn't the issue :/

[16:13:23 CDT(-0500)] <athena> yeah

[16:13:24 CDT(-0500)] <athena> figured

[16:13:27 CDT(-0500)] <athena> super weird.

[16:13:42 CDT(-0500)] <EricDalquist> I'm checking out the resource server

[16:18:35 CDT(-0500)] <athena> ummm . . . huh.

[16:18:35 CDT(-0500)] <athena> ok

[16:18:52 CDT(-0500)] <athena> so i rebuild the resource server with jasig-parent=31 instead of 32

[16:18:56 CDT(-0500)] <athena> then rebuild uportal

[16:18:58 CDT(-0500)] <athena> and it worked

[16:19:26 CDT(-0500)] <EricDalquist> uh

[16:19:40 CDT(-0500)] <athena> yeah that was my response too

[16:21:51 CDT(-0500)] <EricDalquist> so the −32 parent does tweak a bunch of maven plugin versions

[16:21:59 CDT(-0500)] <athena> yeah, looking at that

[16:22:19 CDT(-0500)] <athena> should i just set it to 31 and republish for the moment so we can all move on with the day?

[16:22:28 CDT(-0500)] <EricDalquist> yes

[16:22:30 CDT(-0500)] <athena> k

[16:22:39 CDT(-0500)] <EricDalquist> compiler, jar, war, & release all change

[16:22:40 CDT(-0500)] <EricDalquist> oi

[16:22:52 CDT(-0500)] <EricDalquist> sounds like I get to spend a day debugging that (sad)

[16:22:56 CDT(-0500)] <athena> (sad)

[16:22:58 CDT(-0500)] <athena> no good.

[16:23:18 CDT(-0500)] <athena> yeah that's definitely the bizarre bug of the week . . .

[16:23:28 CDT(-0500)] <holdorph> it's only monday

[16:23:38 CDT(-0500)] <holdorph> do you realize what you're setting yourself up for saying that?

[16:24:26 CDT(-0500)] <EricDalquist> (tongue)

[16:25:12 CDT(-0500)] <EricDalquist> ok I think this is ready: https://wiki.jasig.org/display/UPC/Git+Tips+and+Tricks

[16:25:29 CDT(-0500)] <EricDalquist> I'll run through it all myself tomorrow to verify that

[16:25:52 CDT(-0500)] <EricDalquist> oh I need to add the "pull request" section

[16:27:43 CDT(-0500)] <athena> thanks so much for doing this

[16:27:49 CDT(-0500)] <EricDalquist> no problem

[16:27:55 CDT(-0500)] <athena> will have to try out the suggested branch strategy tomorrow

[16:28:01 CDT(-0500)] <EricDalquist> it will be valuable to have everyone on the same page

[16:28:03 CDT(-0500)] <athena> have to say that's a lot more steps than what we're used to doing

[16:28:12 CDT(-0500)] <EricDalquist> yes

[16:28:16 CDT(-0500)] <EricDalquist> though I'll add some more notes

[16:28:24 CDT(-0500)] <EricDalquist> since a bunch of them are "look at the commit logs"

[16:28:28 CDT(-0500)] <athena> yeah

[16:28:39 CDT(-0500)] <EricDalquist> and you may not need to do them most of the time

[16:28:44 CDT(-0500)] <athena> yeah

[16:28:45 CDT(-0500)] <athena> fair enough

[16:28:48 CDT(-0500)] <EricDalquist> since we don't have 20+ people constantly committing

[16:28:57 CDT(-0500)] <EricDalquist> also all of these do have eclipse equivalents

[16:29:03 CDT(-0500)] <athena> gotcha

[16:29:11 CDT(-0500)] <athena> i haven't succesfully made that work

[16:29:12 CDT(-0500)] <EricDalquist> I figure documenting the CLI is a good lowest common denominator

[16:29:16 CDT(-0500)] <athena> may have to go upgrade eclipse

[16:29:21 CDT(-0500)] <athena> yeah, it is

[16:29:49 CDT(-0500)] <EricDalquist> so as far as I can tell egit doesn't have any understanding of maven nested projects in eclipse

[16:30:01 CDT(-0500)] <EricDalquist> so you have to do all the Team commands on the root folder of the project

[16:30:13 CDT(-0500)] <EricDalquist> it has no idea that "uportal-war" is a git project

[16:30:16 CDT(-0500)] <EricDalquist> (sad)

[16:30:19 CDT(-0500)] <athena> ah.

[16:30:27 CDT(-0500)] <athena> then i guess it works as much as it ever will, in that case (smile)

[16:30:34 CDT(-0500)] <athena> oh well

[16:30:42 CDT(-0500)] <EricDalquist> I guess the bigger problem is that eclipse still has no support for nested projects

[16:30:44 CDT(-0500)] <EricDalquist> which sucks

[16:30:46 CDT(-0500)] <athena> yeah.

[16:33:26 CDT(-0500)] <EricDalquist> ok I have to get going, catch you tomorrow

[16:34:33 CDT(-0500)] <athena> cya!

[16:34:36 CDT(-0500)] <athena> thanks for the help