Log of the #fcrepo channel on chat.freenode.net

Using timezone: Eastern Standard Time
* dhlamb leaves00:14
* dbernstein leaves00:18
* dbernstein joins00:27
* dbernstein leaves02:22
* dwilcox joins04:31
* dwilcox leaves05:21
* dwilcox joins06:05
* dwilcox leaves06:12
* dwilcox joins07:54
* dwilcox leaves
* dwilcox joins08:04
* dwilcox leaves08:30
* dhlamb joins08:51
* acoburn joins08:56
* mikeAtUVa joins08:59
* peichman joins
* mikeAtUVa leaves09:04
* amccarty joins09:06
* ajs6f joins09:19
* dwilcox joins09:34
<ajs6f>Does escowles still live in Florida?09:36
* dwilcox leaves09:37
* osmandin joins09:41
* github-ff joins09:44
[fcrepo4] acoburn opened pull request #1109: Add namespaces to the default CND configuration (master...fcrepo-2258) https://git.io/vPZzp
* github-ff leaves
<acoburn>ajs6f: I think he moved to MD
<ajs6f>acoburn: That's good to hear.
* dwilcox joins09:50
* dwilcox leaves09:57
* peichman leaves10:02
* osmandin leaves10:33
* peichman joins10:34
<ruebot>ajs6f: no gopher party?14:44
<acoburn>mohamedar: thanks for getting the logger back up and running14:45
<ajs6f>ruebot: No. Because gopher party sounds like some kind of weird ritual that is part of college fraternity hazing.
dbernstein: Have you used Git much?
<mohamedar>acoburn: np14:46
* peichman leaves14:48
<dbernstein>ajs6f: I haven’t worked so much with forks and pull requests.14:49
obviously I’m open to suggestions for improvement.
<ajs6f>dbernstein: Cool. I was just going to say that the JVM args bit you put into that PR… the way I would have done that would be as a separate commit as you are working. Then you can cherry-pick the commits that actually matter to the ticket in hand. and use the remainder for new tickets.14:50
dbernstein: This is where having a feature branch associated with a ticket helps.
* peichman joins
<ajs6f>dbernstein: You just keep rolling along, doing what you need to do to get the job done, but every time you do something (like add diagnostic tools) that doesn't directly concern the purpose of the branch in hand, add it in a new commit.14:51
dbernstein: Then, at merge time, you can cherry pick and you don't lose any time reverting stuff and you don't lose any (possibly useful for other things) code changes.
<dbernstein>right - I totally hear you. That was laziness on my part - ie not creating a ticket and separate branch. I did commit the jvm args bit in a separate commit.14:54
When you say “merge time” do you mean when I’m pulling changes from master into my local fork??
<ajs6f>dbernstein: No, sorry, I meant merge time = when you want to merge your new stuff back into master.14:55
dbernstein: The last thing you would do before saying "hey, can someone review this?"
<dbernstein>I see - so when I create the pull request.14:57
<ajs6f>dbernstein: right.14:59
dbernstein: Like you have made up a big box of commits, and then out of it you pull just only those that are absolutely needed for the PR, and that's what you send.15:00
dbernstein; the others are "scrap" that you keep around the workshop for other projects. :
<dbernstein>Cool. Thanks for bearing with me on this first contribution. I appreciate the feedback.15:06
* peichman leaves15:12
* peichman joins15:23
* foo_ joins15:29
* foo_ leaves15:32
<ajs6f>dbernstein++ # you're on your way!15:34
<acoburn>ajs6f: my reading of that is that modeshape Sessions _are_ thread-safe15:37
ajs6f: in which case the FedoraSession would also be thread-safe15:38
* dsteelma joins15:39
<ajs6f>acoburn: My reading is that they are, but that "Although not required by the specification," means we need to be careful. I'd rather not rely on it.
<acoburn>ajs6f: I'll remove the statements about thread-safety15:40
* dsteelma leaves15:41
* dsteelma joins
* dsteelma leaves
<ajs6f>acoburn: My general rule with Fedora is "avoid promising anything at all to anyone in any circumstance".15:46
acoburn: But don't hold me to that.15:47
<acoburn>ajs6f: that's a good principle15:49
<ajs6f>acoburn: In principe.15:50
* amccarty leaves16:11
* peichman leaves16:22
* ajs6f leaves16:25
* peichman joins16:27
* dwilcox leaves16:45
* peichman leaves16:57
* acoburn leaves17:37
* whikloj leaves18:03
* mjgiarlo joins18:24
* mohamedar leaves18:42
* mjgiarlo leaves18:49
* amccarty joins19:32
* amccarty leaves20:16
* dbernstein leaves21:59
* dbernstein joins
* dbernstein leaves22:02
* github-ff joins23:17
[fcrepo-camel-tests] dannylamb pushed 2 new commits to master: https://git.io/vPccE
fcrepo-camel-tests/master 3906ea9 Aaron Coburn: Add httpcore dependency...
fcrepo-camel-tests/master 8b9ff4f dannylamb: Merge pull request #13 from acoburn/fcrepo-2257...
* github-ff leaves
* github-ff joins
[fcrepo-camel] dannylamb pushed 1 new commit to master: https://git.io/vPccu
fcrepo-camel/master e8d2e68 Aaron Coburn: Tighten processor code (#125)...
* github-ff leaves
* travis-ci joins23:21
fcrepo4-exts/fcrepo-camel#295 (master - e8d2e68 : Aaron Coburn): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-camel/compare/0a3ce97733db...e8d2e686cd3e
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-camel/builds/165418039
* travis-ci leaves
* travis-ci joins23:26
fcrepo4-exts/fcrepo-camel-tests#29 (master - 8b9ff4f : dannylamb): The build passed.
Change view : https://github.com/fcrepo4-exts/fcrepo-camel-tests/compare/0f81c4e27af4...8b9ff4ff4ba5
Build details : https://travis-ci.org/fcrepo4-exts/fcrepo-camel-tests/builds/165417982
* travis-ci leaves
* dhlamb leaves23:42
* dbernstein joins23:48
* github-ff joins00:29
[fcrepo-camel] acoburn opened pull request #126: Add a Linked Data Notification processor (master...fcrepo-2254) https://git.io/vPc8C
* github-ff leaves

Generated by Sualtam