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

Using timezone: Eastern Standard Time
* tjohnson leaves05:44
* dhlamb joins08:24
* jcoyne joins08:36
* jrgriffiniii joins09:00
* jrgriffiniii leaves09:06
* jrgriffiniii joins09:08
* ajs6f joins09:10
* whikloj joins09:14
* peichman joins
<jcoyne>Why is jira so slow.09:15
* jrgriffiniii leaves09:23
* osmandin leaves09:24
* jrgriffiniii joins09:27
* acoburn joins09:29
* tjohnson joins09:31
* peichman leaves09:40
* yinlin joins09:47
* jrgriffiniii_ joins09:52
* jrgriffiniii leaves09:53
* diegopino joins10:11
<ajs6f>awoods: Are you working on the Memento/LDP doc now?10:16
<awoods>ajs6f: no10:17
<ajs6f>awoods: Oh, I see your cursor. I guess you just have a webpage open with it.
And now a penguin
awoods: I don't think (using your language, which seems useful) that we can say "LDP-Rm is created on POST to the LDP-Cv." Instead I think we have to say "LDP-Rm is created on POST to the original LDP-R, which is itself a Memento URI-R".10:20
awoods: I.e. when you create an LDP-R using a header for versioning, it is both an LDP-R and a Memento URI-R.
Now there is a llama, too.10:21
<awoods>ajs6f: shall we turn the conversation onto the doc itself?
<ajs6f>awoods: Good call.
* barmintor joins10:48
* barmintor leaves
* bseeger joins10:49
* diegopino leaves10:50
* ajs6f leaves10:58
* dwilcox joins11:03
* github-ff joins11:18
[fcrepo4-vagrant] yinlinchen opened pull request #38: Update the fcrepo4-vagrant configuration to download an F4 webapp art… (master...fcrepo-1918) https://git.io/v2sxg
* github-ff leaves
* yinlin leaves11:25
<acoburn>awoods: in working through fcrepo-1924, I noticed these lines: https://git.io/v2shy11:26
<awoods>on a call
* pmurray joins11:30
dwilcox: ping.
<awoods>dwilcox is on a call
<pmurray>awoods: Ah, okay. Are you?11:31
<pmurray>Hmmm...I just wanted to note that the URL to timeanddate.com for the March 14th book club session is off by an hour. I will correct it in the spreadsheet, but wanted to let dwilcox know that the time he signed up for may not be what he expected (if he followed the link).11:32
Odd -- I can't seem to fix it in the Google Spreadsheet either -- will leave note.11:34
* github-ff joins11:39
[fcrepo4] acoburn opened pull request #990: FedoraResource::getChildren() now returns a Stream (master...fcrepo-1924) https://git.io/v2Gfd
* github-ff leaves
* github-ff joins11:52
[fcrepo4] Rarian opened pull request #991: eTags should be Weak eTags (master...FCREPO-1754) https://git.io/v2GLq
* github-ff leaves
* acoburn leaves12:02
* bseeger leaves12:09
<dwilcox>Thanks, pmurray; I will check the spreadsheet12:10
* github-ff joins12:21
[fcrepo4] escowles created jdbc (+1 new commit): https://git.io/v2GC0
fcrepo4/jdbc f9316d7 Esmé Cowles: Adding configuration for PostgreSQL
* github-ff leaves
* bseeger joins12:24
* diegopino joins12:31
<Guest70607>ruebot/whikloj: I just ran into an issue with fcrepo4-base, and fcrepo4-vagrant, which I think arises from the fact that the most recent version doesn't have a provider in atlas.
<whikloj>Guest70607: what is the issue?12:32
* acoburn joins
<ruebot>Guest70607: it has a virtualbox provider12:33
<ajwagner>4.4.0 does, 4.4.1 doesn't.
ajwagner: i'm not sure i'm understanding what you're getting at.12:34
<whikloj>v4.4.1 - There is not currently a released version # Maybe that is the reason
<ajwagner>When I vagrant up, I get a "The box 'fcrepo/fcrepo4-base' could not be found."
There are no providers for this version
<ajwagner>I had to manually run "vagrant box add fcrepo/fcrepo4-base https://atlas.hashicorp.com/fcrepo/boxes/fcrepo4-base/versions/4.4.0/providers/virtualbox.box"
In order to get it to work.12:35
I'm just guessing... that it may be related to the online install wanting the latest version, and that not actually having a provider.
*and that version not actually...
<ruebot>ajwagner: i'll have to talk to awoods. iirc, i only ever created 4.4.0. not sure what 4.4.1 is.12:36
<ajwagner>Okay, just figured I'd mention it, in case other people have issues :)12:37
<ruebot>ajwagner: i think you're the first. i haven't seen it before.12:41
ajwagner: what version of vagrant are you using, and are you using any vagrant plugins?
<ajwagner>I don't think it's an issue with my vagrant setup, but I didn't rule it out.
ruebot: 1.4.3, with no plugins.12:42
* travis-ci joins12:45
fcrepo4/fcrepo4#4293 (jdbc - f9316d7 : Esmé Cowles): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/commit/f9316d7f0788
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/111264322
* travis-ci leaves
* github-ff joins12:47
[fcrepo4] awoods pushed 1 new commit to master: https://git.io/v2Ggf
fcrepo4/master 71c568c Andrew Woods: Merge pull request #986 from escowles/jdbc...
* github-ff leaves
<ruebot>ajwagner: that's pretty old. can you update?12:50
<awoods>ruebot/ajwagner: vagrant destroy / vagrant up worked for me on "master"12:55
<ajwagner>Ahh okay, I'll presume it's just because I'm using the version of vagrant from ubuntu 14.04. Nvm.12:56
1.8.1 does function as expected.12:59
<ajwagner>I'm about to run into some meetings but I'll send a PR for the readme, that lists the requirement for 1.5+, as the shorthand name rather than full url was introduced then.13:03
* travis-ci joins13:07
fcrepo4/fcrepo4#4295 (master - 71c568c : Andrew Woods): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/34b50e13f249...71c568cf19c8
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/111270459
* travis-ci leaves
* github-ff joins13:19
[fcrepo4] acoburn opened pull request #992: change property2values function to return a Stream (master...fcrepo-1926) https://git.io/v2G9A
* github-ff leaves
<awoods>whikloj: do you happen to remember why you removed "fcrepo-audit-triplestore" from fcrepo4-vagrant? https://github.com/fcrepo4-exts/fcrepo4-vagrant/commit/0c6e9a13311bc9c4d7e881367b5bfef0a619e08e13:20
* diegopino leaves13:28
* acoburn leaves13:31
* github-ff joins13:37
[fcrepo4-vagrant] awoods pushed 2 new commits to master: https://git.io/v2Gd3
fcrepo4-vagrant/master 5544e33 Yinlin Chen: Update the fcrepo4-vagrant configuration to download an F4 webapp artifact that contains /fcr:transform but not audit nor auth.
fcrepo4-vagrant/master 89a8c78 Andrew Woods: Merge pull request #38 from yinlinchen/fcrepo-1918...
* github-ff leaves
* acoburn joins13:44
* ajs6f joins13:55
* jgpawletko leaves14:26
* jgpawletko joins14:57
* jgpawletko leaves
<ajs6f>awoods: The code style settings are set to disallow: private Type t1,t2,t3; (Multiple declarations in a single statement.) I think we should allow that. It's often _more_ clear and concise, especially in unit tests where you're setting up a bunch of almost identical objects in a frame.15:03
<awoods>ajs6f: I believe the dev team advocated against multiple declarations in one line.15:05
<ajs6f>awoods: I don't remember that at all. Even if so, I am reopening the question.
awoods: I'm not suggesting that we make mass edits to the codebase. I'm suggesting that we remove the restriction.15:06
<awoods>ajs6f: personally, I find it more clear to have declarations on separate lines... but am happy to go with consensus.15:08
<ajs6f>awoods: Do we have a standard way to poll our ambiguously-defined and ill-bounded dev army?15:09
<awoods>ajs6f: I suppose polling fedora-committers would to the trick.
<ajs6f>awoods: +115:10
<awoods>ajs6f: These sort of either/or questions should probably not rely on lazy-consensus
<ajs6f>awoods: What are you suggesting?15:11
3 +1 no -1?
<awoods>ajs6f: what is 3?
<ajs6f>awoods Three people voting +1, no people voting -1.
awoods: Although with our small and disengaged pool, maybe two +1 no -1 is better.15:12
<awoods>ajs6f: I would suggest something like: vote for either single-line declaration or multiple-line declarations... and a majority is required for one or the other.15:13
<ajs6f>awoods: Majority of the committers?
<awoods>ajs6f: yes
<ajs6f>awoods: No. That's a recipe for nothing ever happening.
awoods: I don't understand what's wrong with lazy consensus here, anyway,15:14
<awoods>ajs6f: lazy consensus is biased towards the change
<ajs6f>awoods: Which is a problem why?
<awoods>ajs6f: because people care on both sides of the question (specifically, me and you)
<ajs6f>awoods: Then you have the right to -1. That's the whole point.15:15
awoods: Demanding to hear from a majority of committers is obviously biased against the change.
<awoods>ajs6f: let's try the non-lazy approach and expect the best from the committers15:16
<ajs6f>awoods: No. That's as much as to disallow the change. That's a waste of my time.15:17
<awoods>ajs6f: if no consensus is formed, we can fall back on a lazy approach
<ajs6f>awoods: {sigh} All right.15:18
awoods: I'm also going to suggest some additions for lambdas, which aren't addressed.15:19
<awoods>ajs6f: that would be great... checkstyle now supports lambdas?
<ajs6f>awoods: I don't know. Just because policy is not mechanically actionable doesn't mean we can't define it. It's just that we may miss some things until CS catches up, at which time we can make any corrections we need to.15:20
<awoods>ajs6f: I doubt there will be debate on adding new checkstyle for lambdas. Naturally that will be a separate discussion from the single/multi line declaration proposal.15:22
<ajs6f>awoods: There may be debate on the content of lambda style guides. I've seen several different ways of writing them.15:23
<awoods>ajs6f: one way to find out15:24
<ajs6f>awoods: Right. Write a question on a piece of paper, put it in a bottle, throw it into the heart of the Sun. I'm on it.15:25
* acoburn leaves15:28
<awoods>ajs6f +115:31
* dwilcox leaves15:45
* dwilcox joins16:00
* dhlamb leaves16:02
* dhlamb joins
* dhlamb leaves16:07
* peichman joins16:21
* bseeger leaves16:46
* peichman leaves16:55
* whikloj leaves17:00
* ajs6f leaves17:01
* dwilcox leaves17:05
* jcoyne leaves17:08
* bseeger joins17:19
* bseeger leaves17:20
* jgpawletko joins17:25
* jgpawletko leaves17:26
* dwilcox joins17:40
* dwilcox leaves17:53
* pmurray leaves17:58
* jrgriffin_away leaves
* escowles leaves
* tjohnson leaves
* anarchivist leaves
* ajwagner leaves
* awoods leaves
* petertr7_away leaves
* f4jenkins leaves
* ruebot_away leaves
* petertr7_away joins18:01
* f4jenkins joins
* awoods joins
* escowles joins
* jrgriffin_away joins
* ajwagner joins
* anarchivist joins
* tjohnson joins
* pmurray joins18:03
* ruebot_away joins18:04
* dwilcox joins18:06
* mcritchlow joins18:08
* dwilcox leaves18:11
* github-ff joins18:21
[fcrepo-module-auth-webac] awoods opened pull request #56: Change FedoraResource.getChildren() to return Stream (master...fcrepo-1924) https://git.io/v2ZpB
* github-ff leaves
* github-ff joins18:24
[fcrepo4] awoods pushed 1 new commit to master: https://git.io/v2Zpp
fcrepo4/master 85ad606 Aaron Coburn: FedoraResource::getChildren() now returns a Stream...
* github-ff leaves
* github-ff joins
[fcrepo-module-auth-webac] awoods pushed 2 new commits to master: https://git.io/v2ZhU
fcrepo-module-auth-webac/master 0ca2943 Andrew Woods: Change FedoraResource.getChildren() to return Stream...
fcrepo-module-auth-webac/master d1c7ba6 Andrew Woods: Merge pull request #56 from awoods/fcrepo-1924...
* github-ff leaves
* github-ff joins18:25
[fcrepo4] awoods closed pull request #990: FedoraResource::getChildren() now returns a Stream (master...fcrepo-1924) https://git.io/v2Gfd
* github-ff leaves
* jrgriffin_away leaves18:28
* travis-ci joins18:29
fcrepo4/fcrepo-module-auth-webac#103 (master - d1c7ba6 : Andrew Woods): The build has errored.
Change view : https://github.com/fcrepo4/fcrepo-module-auth-webac/compare/3a6f435e033f...d1c7ba61651e
Build details : https://travis-ci.org/fcrepo4/fcrepo-module-auth-webac/builds/111350601
* travis-ci leaves
* travis-ci joins18:43
fcrepo4/fcrepo4#4300 (master - 85ad606 : Aaron Coburn): The build passed.
Change view : https://github.com/fcrepo4/fcrepo4/compare/71c568cf19c8...85ad60670624
Build details : https://travis-ci.org/fcrepo4/fcrepo4/builds/111350543
* travis-ci leaves
* dwilcox joins18:45
* dwilcox leaves18:54
* mcritchlow leaves19:33
* dhlamb joins20:22
* jcoyne joins20:51
* dhlamb leaves22:40
* mcritchlow joins22:55
* tjohnson leaves23:14
* jcoyne leaves23:48
* mcritchlow leaves00:05