-
Tarun_0x0 joined the channel
-
reosarevok
yvanzo: huh, still can't locate
-
yvanzo
reosarevok: Is the environment variable really set? `docker-compose run --rm musicbrainz bash -c env | grep CARTON`
-
reosarevok
-
Seems not
-
yvanzo: do you think the no cache would help?
-
yvanzo
reosarevok: no, do you really have the latest force-pushed changes?
-
reosarevok
-
log suggests so
-
yvanzo
It seems so. You don’t have local changes either? `git grep CARTON`
-
reosarevok
-
yvanzo
Is there a running musicbrainz service atm? `docker compose ps musicbrainz`
-
Your Git working tree seems to be fine, but the Docker image used in your run command doesn’t have this environment variable for some reason.
-
reosarevok
Seems not...
-
I need to do up before this?
-
yvanzo
No, quite the contrary, I thought that it might be the cause of your issue.
-
reosarevok: Did you try again? `docker compose build musicbrainz && docker compose run --rm musicbrainz createdb.sh`
-
reosarevok
I'm going out but I'll leave it running
-
Huh
-
It does seem to be working this time
-
What changed?
-
yvanzo
I don’t know. Maybe your Git working tree wasn’t up-to-date before building the image again?
-
Or the previous build failed for some reason?
-
Tarun_0x0 has quit
-
bitmap: I just read that your PR is actually requiring PG 16.
-
Does the other PR require it too?
-
bitmap
yes, I think prior to PG16 the INHERIT option must be added to the role itself, affecting all GRANTs
-
which may be a better option in any case
-
the RO connector one does not require PG16. (I also have to update the PR description)
-
yvanzo
Ok, I will look into it after a break.
-
v6lur_ joined the channel
-
bitmap
the latter could also be delayed until a later release if needed
-
(the privileges one is intended to run in upgrade.sh though)
-
v6lur_ has quit
-
yvanzo: actually I don't think WITH INHERIT is needed since roles have the INHERIT flag by default (which affects GRANTs). so I'll just remove it
-
lucifer
-
mayhem
monkey recently switched to linux. :/
-
lucifer
oh. anyone else with a mac?
-
mayhem
but I think his old laptop might still work. not sure if outsidecontext has access to a mac.
-
outsidecontext
I have an old Mac mini. I can check on that
-
mayhem
👍
-
akshaaatt
-
lucifer ^
-
lucifer
akshaaatt: awesome, thanks!
-
did it install like a normal macos app?
-
or did you have to do any additional or unusual steps?
-
akshaaatt
Yus it did
-
No additional steps
-
lucifer
great!
-
reosarevok
yvanzo: loading url now - after this is done, I guess what, just load the site and see if it works?
-
outsidecontext
lucifer: works here as well on macos 12.7. also the appimage runs on my linux lapttop
-
mayhem
lucifer: appimage works on my linux install too.
-
aerozol[m] joined the channel
-
aerozol[m] uploaded an image: (14KiB) < https://matrix.moviebrainz.org/_matrix/media/v3/download/matrix.org/hhuoeABTMlxWcQXOLNJoKjYM/image.png >
-
aerozol[m]
akshaaatt: oh no, any further tips? ^
-
reosarevok
yvanzo: finally all done but it's late so will test in the morning
-
aerozol[m]
lucifer: installed fine on Win 10
-
Didn’t ask where to install etc but I don’t usually install ‘apps’ so that might be normal?
-
Has an error on opening but I assume we’re just testing the install
-
kilos_8 joined the channel
-
kilos_ has quit
-
kilos_8 is now known as kilos_
-
Kladky has quit
-