but I can't find that in the current version of the readme
yvanzo: ^ is 1.21 still needed? if we can just fix the commands to use the correct names instead that'd be a lot less annoying for people using ubuntu 18.04 I think
nelgin
I can try 1.17.1 and see.
bitmap
if you run into an issue with the names maybe we can find a workaround
not sure that's a good reason to require people to upgrade at least
nelgin
It's at the "setting up" stage
Unescaped left brace in regex is deprecated, passed through in regex; marked by <-- HERE in m/(?<!\\)\${ <-- HERE ([^}]+)}/ at /usr/sbin/pam_getenv line 78.
Got a bunch of those after Setting up postgresql
invoke-rc.d: could not determine current runlevel
invoke-rc.d: policy-rc.d denied execution of start.
Some runlevel stuff
bitmap
does the whole process abort after those message, or does it keep going? (wondering if they're non-fatal warnings that can be ignored)
nelgin
Keep going
It's building the perl modules now
bitmap
ok, maybe they're harmless then. I've seen the rc.d ones before and the regex one looks non-fatal
(I haven't used musicbrainz-docker before, but I've seen them elsewhere)
nelgin
arning about asset size..
yarn install v1.19.1
[1/4] Resolving packages...
[2/4] Fetching packages...
info fsevents@1.2.7: The platform "linux" is incompatible with this module.
info "fsevents@1.2.7"; is an optional dependency and failed compatibility check. Excluding it from installation.
info fsevents@1.2.4: The platform "linux" is incompatible with this module.
info "fsevents@1.2.4"; is an optional dependency and failed compatibility check. Excluding it from installation.
A bit further up I got these
Creating the database, guess I'll check in tomorrow lol
bitmap
heh, okay
those you can ignore too
alright, finished the mbs release now
nelgin
databas eimported. working on creating indexes
adiiiiiiiiiiiiii
sbvkrishna: Hey guys, i got my GitHub account back! It was a random flag done by the GitHub spam control system. Anyways my PR is still up https://github.com/bookbrainz/bookbrainz-site/p.... Could anyone check into it?
Also can i too contribute to the Google Code-In as a mentor ?
I was a code-in participant for 2018 and 2017 and have done some task with Musicbrainz and bookbrainz.
chaban has quit
chaban joined the channel
pristine__
iliekcomputers: hi
nelgin
Ok, I'm at InitDB.pl succeeded. Let's see what's next.
Docker is up. Let's see if I can build the search indexes. I'm running on a vm with 2 CPUs and 4GB of memory but I'm using a fixed rather than dynamic disk...so maybe take a day or two :)
musicbrainz@musicbrainz:~/musicbrainz-docker$ sudo docker-compose exec indexer python -m sir reindex
As an alternative, you can either use COMPOSE_PROJECT_NAME=musicbrainz-docker (and rebuild every image so far) or replace musicbrainz-docker_ with musicbrainzdocker_ in this file.
reosarevok: it is already consistent, with 'any' :)
reosarevok
I mean, that to me is like saying "it's already consistent if we turn the rule off"
yvanzo
reosarevok: this rule is here to spot common mistakes, 'any' still detects such mistakes.
reosarevok
But why would we choose to have our code be less standardised if we can make it more standardised?
yvanzo
No, turning the rule off wouldn't detect such error.
reosarevok
It feels super weird to me to choose to have one thing that can be done in two ways done in different ways throughout the code
nelgin
This is way too complicated for someone who justs wants to get it up and running. I've been doing unix/linux admin for over 30 years and I'm still having problems following along.
yvanzo
nelgin: I agree, sorry, it is currently a bit of a mess, due to the ongoing search server migration, plus the schema change.
nelgin
So it'll really fail if I try and build/run it with docker-compose 1.17.1 even tho that's the latest that comes with Ubuntu?
yvanzo
Yes, not because of features, because it has a bug that is fixed in 1.21.0-rc1 only.
Note, it used to work with older versions of docker-compose before it broke stuff.
It still is probably less pain than handling dependencies, pg extensions, and all by hand.
nelgin
Oh well...
At least I have the database, I'll just write some perl scripts to extract what I need rather than having the fun api.
Nyanko-sensei has quit
Why not just write it in php and use the latest mysql and be done with it. Everyone has php & mysql
or mariadb anyway, who the hell uses postgres any more ;)
D4RK-PH0ENiX joined the channel
reosarevok
Because then we'd have to write PHP
(I think that's the answer you'll get from all our devs, not just me :p )
And I mean, there's really not that much of a difference between mysql and postgres for just using the data anyway. I suspect it'd be as complicated in mysql/mariadb, it's just a complicated DB with a lot of things going on
ruaok
moin!
ruaok shudders at the thought of using PHP or even going back to the red hot mess that is mysql
bitmap
yvanzo: can't you explicitly set the project name to be without the hyphen though?
yvanzo
bitmap: yes but then you would have to set COMPOSE_PROJECT_NAME for recent versions...
bitmap
I mean the readme would have to include -p or whatever in all the commands so it'd work in either version
yvanzo
...which is probably more pain than installing latest version from dedicated apt repos.
bitmap
I don't see how making people to upgrade is less painful tbh, since they're copy/pasting the compose commands from the readme anyway
loujine joined the channel
or we could provide scripts for that
yvanzo
Ultimately, there should be a global script indeed.
But if we change project name, that would break current users setup on git pull.