#musicbrainz-devel

yeh we are hitting the rate limiter in our testing, and offering to mirror

The current reosarevok_ server has release events containing just date and area, NO country

that sounds correct

as reosarevok_ is running the areas branch

Right, so both you and me need to make changes to use area instead of country

I don't need to make any changes, as ian is handling the merge

ok, unfortunately I do

so that point you made about date should come before country is kind of moot, as there dis no longer country

ijabz: true

but it was invalid within my branch

Of course it failed just when I changed my name

Anyway

ijabz, http://reosarevok.mbsandbox.org/ws/2/area/db322... might interest you

Added

i work from tickets

The search server isn't in git yet, right?

not least because it provides a place to spec out the requests, discussions etc

Anyway, first I should fix all these tests

no, thank goodness

reosarevok: heh, never heard it that way round before :)

Okay, but if you want to do search server ping me, I can guide you through, its really pretty easy

ok! :) I'll finish the broken bits here first

But after that I might try :)

ocharles: well, that's what I'm used to :)

maybe svn is easy, but it looks kinda intimidating

iirc

trust me if yo can use git you can use svn, anyway the exact commands you need for search server ar win the readme

huh

http://musicbrainz.org/doc/Release#Status is also cut

(on the tests themselves)

"

meeeh

hola!

hola warp :)=

Oh, so I need to add ids, huh :)

(or fix the ids or something)

add ids?

to the html to fix the for thingies

ids are not usually mandatory in html :). (though they are often useful for our userscripting friends)

ah, for clicking <label>s

yep

meh, /ws/2 doesn't give you enough information to actually display a relationship

at least, I don't think there's a way to get a verbose link phrase out of it

I think nikki did complain about how sucky ws/2 is for rels, yeah

(when trying to do stuff with it a while ago)

I wanted to use /ws/2 to search for example relationships

I may continue to do so, just with a crappy UI - but the UI is for relationship editors anyway so I don't mind shipping something a little rough

burrito time now though

ocharles would that be http://tickets.musicbrainz.org/browse/SEARCH-179 feel free to comment on ticket

niiice, now it passes :)

function instead" all the time

reosarevok: hm, I haven't seen that. perhaps that was recently added to Catalyst.

I mean, they work

they just say that over and over

sure

D:

he said emacs

"over and over" reminds me of http://youtu.be/9VDvgL58h_Y

hah

It's been a while since I saw that :)

https://github.com/ku-fpg/sunroof-examples/blob... sunroof looks exciting

what does it do?

lets you write JavaScript in Haskell, by providing a monad that can be reified and compiled to javascript

which is a bit different from the competition, which either try and parse haskell, or literally compile haskell to js

So now we have blaze-html, clay and sunroof to build web applications in Haskell, all sharing code and types

and aeson for json

ok, that sounds interesting.

(I need more time per day)

if you find some and have too much, i'll take some of that

and me

:)

ocharles: https://bitbucket.org/ocharles/musicbrainz-serv...

Oh, nvm :)

that *should* be fine

Oh, I see now :)

heh, keeps failing but with Can't locate object method "load_with_tracklist_for_recording" now

that sounds like my area of breaking stuff

I guess we want load_with_medium_for_recording now

heh

DBD::Pg::st execute failed: ERROR: relation "tracklist" does not exist

fiine, not ready yet

reosarevok: where/how did you get that? (so I can fix it)

Wait, I'm pulling in case it's fixed

nope

:)

There are a few tests

hm, t/rdfa/recording/recording-rdfa.t seems to be the only one

and perhaps the pgtap things, but I don't know exactly how that stuff works.

the others seem unused

Well, if unused then we should remove them, right? :)

This is the t/lib/t/MusicBrainz/Server/Data/Release.pm one I think

yes.

Data::Release passes here

huh

(here being mbs-6023-master in my account on bitbucket)

I just merged that one in

And it still fails here

But it fails in more places so I'll start with those

Maybe that'll magically fix it :p

hrm, I don't know how the t/rdfa/* tests run either.

huh

t::MusicBrainz::Server::Edit::Label::Edit

You must specify isni_codes at lib/MusicBrainz/Server/Edit/Role/ValueSet.pm line 18, <FILE> line 6.

reosarevok: I think I've done enough release editor testing for now. anything I can do to help with what you're doing?

warp: maybe. Try running t::MusicBrainz::Server::Edit::Release::AddCoverArt

on which branch?

Er, dunno, wherever you have your changes for file types

reosarevok: ok

warp: I expect that just requires adding the same line I added to caa.sql to some other file, really

(the INSERT INTO cover_art_archive.image_type (mime_type, suffix) VALUES ('image/jpeg', 'jpg'); ) thingy

but unsure where

reosarevok: possibly, though there may be some fussing with "image/jpeg" not being default for one of the columns.

dunno. if you fix it, I'll be happy :)

(I wanted it default, ocharles didn't. I think ocharles made a ticket for it, but I cannot quickly find it now)

well I'd never seen it. agai nand agai nandagain. wtf :D

ocharles: when you have some time, could you see if t::MusicBrainz::Server::Edit::Label::Edit passes in your ISNI branch in isolation?

warp: A ticket for it not being default?L

I asked for it to not be the default until I gave you an approval on the PR

ocharles: I was thinking of http://tickets.musicbrainz.org/browse/MBS-6133

ah

ocharles: the approval was for the version where image/jpeg is not default