[13:52:33]
<Tag> because versions should be incremental (sorry for the short answer, i'm runing out of time right now)
[13:52:36]
<Tag> > <@lsolrac:matrix.org> Sorry to bother, but I'm a bit lost... apt suddenly fails? https://ci-apps-dev.yunohost.org/ci/job/9352 Tried it 3 times too, no changes to the packages required. Onlyu change this commit were some suggestions from the YunoRunner and correcting the systemd execStart command
Hi there o/ The only change that I think could messed up with apt is the version. I suggested a change on your PR
[13:52:38]
<Solrac> > <@tag:lostpod.me> Hi there o/ The only change that I think could messed up with apt is the version. I suggested a change on your PR
Oh I didnt think the cpe id would be enough.
Though why the (comit) to 0? 🤔 I think short hashes would be better than long ones, but would 0 be good enough?
[13:54:38]
<Solrac> point*
[13:54:38]
<Solrac> fair poit
[14:00:59]
<Salamandar> https://github.com/YunoHost-Apps/ihatemoney_ynh/pull/53/files#diff-6f67d3a8330a3c924847c4e3e44fa893ed4178fee92b9bfeba68fd6fdbaff5ecR46-R62
[14:11:18]
<Yunohost Git/Infra notifications> [apps] @Salamandar edited [pull request #1725](https://github.com/YunoHost/apps/pull/1725): Add schema URLs via autopatch and convert_app_to_manifestv2
[14:11:27]
<Yunohost Git/Infra notifications> [apps] @alexAubin merged [pull request #1725](https://github.com/YunoHost/apps/pull/1725): Add schema URLs via autopatch and convert_app_to_manifestv2
[14:11:28]
<Yunohost Git/Infra notifications> [apps] @alexAubin pushed 3 commits to master ([2ebfdd1e16e0...b3ba5b3ba4b4](https://github.com/YunoHost/apps/compare/2ebfdd1e16e0...b3ba5b3ba4b4))
[14:11:28]
<Yunohost Git/Infra notifications> [apps/master] Add autopatch to add schema to toml files - Félix Piédallu
[14:11:29]
<Yunohost Git/Infra notifications> [apps/master] Merge pull request #1725 from Salamandar/autopatch-schema Add schema URLs via autopatch and convert_app_to_manifestv2 - Alexandre Aubin
[14:11:29]
<tituspijean> I think I never used the autopatch.py, instead I reinvented the wheel with the PyGithub lib
[14:11:29]
<Yunohost Git/Infra notifications> [apps/master] Add manifest URL in conversion script manifestv1 -> manifestv2 - Félix Piédallu
[14:11:30]
<tituspijean> But I can try running it from home tonight
[14:11:40]
<Aleks (he/him/il/lui)> maybe titus knows better about it ? I think at some point there was a patch for $http_host etc in nginx
[14:12:54]
<Aleks (he/him/il/lui)> regarding the autopatch thingy : i don't know who does run the autopatch, i had trouble last time with it but it was a while ago, like 1 year ago
[14:12:56]
<Aleks (he/him/il/lui)> (everytime I see your name, for some reason I think about Shaolin Soccer /o\)
[14:13:05]
<Yunohost Git/Infra notifications> [gitlab_ynh] @Plumeros opened [issue #229](https://github.com/YunoHost-Apps/gitlab_ynh/issues/229): Error installingGitLab
[14:13:05]
<Yunohost Git/Infra notifications> App ntfy rises from level 1 to 7 in job [#18116](https://ci-apps.yunohost.org/ci/job/18116) !
[14:13:05]
<Aleks (he/him/il/lui)> > <@titus:pijean.ovh> I think I never used the autopatch.py, instead I reinvented the wheel with the PyGithub lib
ah well yes my autopatch code is horrendous, definitely should be converted to the pygithub lib ...
[14:13:05]
<Yunohost Git/Infra notifications> App qr rises from level 6 to 8 in job [#18213](https://ci-apps.yunohost.org/ci/job/18213) !
[14:13:50]
<tituspijean> https://forum.yunohost.org/t/ghost-blogging-platform/5332/92
Ghost will definitely break at some point because they do not support MariaDB :/
[14:13:50]
<tituspijean> > <@Alekswag:matrix.org> ah well yes my autopatch code is horrendous, definitely should be converted to the pygithub lib ...
But my solution is not quite elegant either, plenty of lines are copied from a script to another
[14:13:53]
<Aleks (he/him/il/lui)> Yeah i'm even surprised it stills sort of works
[14:13:53]
<tituspijean> Yup
[14:13:53]
<Aleks (he/him/il/lui)> And from what i remember they dont support postgres either, only fucking mysql ?
[14:13:54]
<Aleks (he/him/il/lui)> Nope
[14:13:54]
<Aleks (he/him/il/lui)> That's just crazy, like how to shoot yourself in the foot 101 : require stuff not available in Debian
[14:13:54]
<Aleks (he/him/il/lui)> Nope nope nope
[14:13:55]
<Yunohost Git/Infra notifications> [package_linter] @Salamandar opened [pull request #117](https://github.com/YunoHost/package_linter/pull/117): Fix package linter on apt list
[14:13:55]
<Aleks (he/him/il/lui)> That's Cyberfrankensteinhost, not Yunohost
[14:13:55]
<Yunohost Git/Infra notifications> @Salamandar forked package_linter to [Salamandar/package_linter](https://github.com/Salamandar/package_linter)
[14:13:55]
<tituspijean> Their justification hits home though.. "we are a small team with limited resources" 😅
[14:13:56]
<tituspijean> Hence my $1B question about specifically supporting MySQL during the camp 😅
[14:13:56]
<tituspijean> eh no sorry, v5. v8 is in sid.
[14:13:56]
<tituspijean> and v8 is the one supported by Ghost
[14:13:56]
<tituspijean> > <@Alekswag:matrix.org> That's just crazy, like how to shoot yourself in the foot 101 : require stuff not available in Debian
I did not understand this though, `mysql-common` is an apt package available in Debian :o
[14:14:24]
<Yunohost Git/Infra notifications> [package_linter] @alexAubin pushed 2 commits to master ([5d90885b49cb...c6cbd26235b3](https://github.com/YunoHost/package_linter/compare/5d90885b49cb...c6cbd26235b3))
[14:14:24]
<Aleks (he/him/il/lui)> like there's a legacy thing that `systemctl status mysql` is an alias for `mariadb` etc
[14:14:24]
<Aleks (he/him/il/lui)> my understanding is that it might be a mapping to mariadb ?
[14:14:24]
<Yunohost Git/Infra notifications> [package_linter] @alexAubin merged [pull request #117](https://github.com/YunoHost/package_linter/pull/117): Fix package linter on apt list
[14:14:24]
<Yunohost Git/Infra notifications> [package_linter] @alexAubin approved [pull request #117](https://github.com/YunoHost/package_linter/pull/117#pullrequestreview-1613352260) Fix package linter on apt list
[14:14:25]
<Yunohost Git/Infra notifications> [package_linter/master] Merge pull request #117 from Salamandar/fix_apt_linter Fix package linter on apt list - Alexandre Aubin
[14:14:25]
<Yunohost Git/Infra notifications> [package_linter/master] Allow packages to be a list. Also, use the same splitting logic as in yunohost source code. - Félix Piédallu
[15:53:24]
<Aleks (he/him/il/lui)> > <@titus:pijean.ovh> Their justification hits home though.. "we are a small team with limited resources" 😅
yah and we're a small team with limited resources too, we ain't gonna submit to every project weird specificities x_x "Project X wants mysql, project Y wants Python 3.42, project Z wants Python 2.4", like, nope
[15:53:25]
<Aleks (he/him/il/lui)> or we rewrite YunoHost using a full dockerized setup and require at least 4GB of RAM x_x
[16:04:32]
<orhtej2> > <@Alekswag:matrix.org> or we rewrite YunoHost using a full dockerized setup and require at least 4GB of RAM x_x
Come to think about it a lot of apps ship docker containers and make standalone install super convoluted
[16:04:57]
<orhtej2> Otoh do you want to run 222666 docker containers side by side
[16:05:54]
<orhtej2> Like duplicates of postgresql I mean
[16:24:48]
<Yunohost Git/Infra notifications> App mybb goes down from level 8 to 6 in job [#18295](https://ci-apps.yunohost.org/ci/job/18295)
[16:39:46]
<Solrac> Hello again ^^"
I think I'm starting to be a bit confused. I'm certain the error now lies with MotionEye, so I wanted to ask, if there's way to manually test a failed install to see what's up. However one of the tests passed? https://ci-apps-dev.yunohost.org/ci/job/9358
[16:43:02]
<Solrac> I feel a bit silly looking at the permission error now
[16:52:50]
<Yunohost Git/Infra notifications> App seafile failed all tests in job [#18297](https://ci-apps.yunohost.org/ci/job/18297) :(
[17:08:45]
<selfhoster1312> eric_G, je comprends pas: https://github.com/YunoHost-Apps/invidious_ynh/blob/master/scripts/install#L77
on appelle un invidious déjà compilé précédemment ? il me semble qu'il est compilé après cette ligne oO
[17:14:11]
<eric_G> ¯\_(ツ)_/¯
[17:53:44]
*selfhoster1312 va réinstaller invidious et lire le log attentivement :D
[18:08:17]
<selfhoster1312> mais... y'a déjà un `make` ligne 53, je rêve où invidious est compilé deux fois par install? oO
[18:13:18]
<selfhoster1312> du coup le deuxième build même sur ma machine patate il met quelques minutes de plus ^^
[18:20:15]
<eric_G> we never know with compilation... 😅
[18:21:21]
<eric_G> but seriously, I may have done crap, so feel free to push a PR
[18:21:41]
<selfhoster1312> i'll check it out thanks :)
[18:39:39]
<selfhoster1312> anecdotally it works on my server ®
[18:39:48]
<selfhoster1312> and the CI is running: https://ci-apps-dev.yunohost.org/ci/job/9373
[18:52:12]
<Solrac> What can I do for permissions? I'm doing the chown $app:www-data **and** chmod 777 (which doesnt feel right) for the directories, yet the service continues to say that /etc/motioneye doesn't exist or can't be written to, despite mkdir -p before the pushd is called for the actual install 🤔
https://ci-apps-dev.yunohost.org/ci/job/9372
[19:13:06]
<selfhoster1312> when you actually install the app on a real server what are the final permissions on /etc/motioneye?
[19:14:38]
<orhtej2> > <selfhoster1312> when you actually install the app on a real server what are the final permissions on /etc/motioneye?
perhaps just change [`conf_path`](https://github.com/YunoHost-Apps/motioneye_ynh/blob/3223239d33af0e3465682245f667c42c2f091e68/conf/motioneye.conf#L2) to subdir of `$install_path`?
[19:16:58]
<orhtej2> also [this](https://github.com/YunoHost-Apps/motioneye_ynh/blob/3223239d33af0e3465682245f667c42c2f091e68/conf/motioneye.conf#L11) should likely point to `__DATA_DIR__`
[19:17:36]
<Solrac> > <selfhoster1312> when you actually install the app on a real server what are the final permissions on /etc/motioneye?
Lemme try installing on a lxc to confirm, cause Im no longer sure
[19:18:55]
<selfhoster1312> eric_G, looks like tests passed and took "only" 46 minutes vs 1h3m for the last CI run for invidious package
[20:09:04]
<eric_G> I don't understand this warning:
`! When using a postgresql database, you should add postgresql in apt dependencies. ` in this test: https://ci-apps-dev.yunohost.org/ci/job/9374
[20:30:06]
<selfhoster1312> eric_G:
```
apt_packages = resources["apt"].get("packages", [])
if isinstance(apt_packages, str):
apt_packages = [value.strip() for value in apt_packages.split(",")]
```
[20:30:44]
<selfhoster1312> so apparently it should be a key called "packages" in apt, and should be split by ","
[20:30:49]
<selfhoster1312> maybe some manifestv2 thing?
[20:31:54]
<selfhoster1312> eric_G, see https://github.com/YunoHost/example_ynh/blob/master/manifest.toml#L117C28-L117C28
[20:32:41]
<selfhoster1312> but personally i'd rather go with a proper toml list :) :) :)
[20:32:46]
<selfhoster1312> (since the linter says it's ok)
[20:41:07]
<orhtej2> > <@ericg:matrix.org> this warning is strang... I don't think the comma is the culprit. https://github.com/YunoHost-Apps/invidious_ynh/blob/3eba9b5076feb11d2bfbe0efbec1ef88bd0bbe4e/manifest.toml#L68
the piece of code selfhoster1312 linked will turn "apt1 apt2" into list of ['apt1 apt2'] rather than ['apt1', 'apt2'], hence i.e. apt1 is not on the list
[20:41:07]
<eric_G> this warning is strang... I don't think the comma is the culprit. https://github.com/YunoHost-Apps/invidious_ynh/blob/3eba9b5076feb11d2bfbe0efbec1ef88bd0bbe4e/manifest.toml#L68
[20:41:07]
<orhtej2> > <@ericg:matrix.org> this warning is strang... I don't think the comma is the culprit. https://github.com/YunoHost-Apps/invidious_ynh/blob/3eba9b5076feb11d2bfbe0efbec1ef88bd0bbe4e/manifest.toml#L68
ALL of these dependencies from linters standpoint are singular dependency of stupid name ;P
[20:42:43]
<Yunohost Git/Infra notifications> App glitchsoc rises from level 6 to 7 in job [#18302](https://ci-apps.yunohost.org/ci/job/18302) !
[20:44:35]
<Yunohost Git/Infra notifications> @orhtej2 forked package_linter to [orhtej2/package_linter](https://github.com/orhtej2/package_linter)
[20:48:34]
<Yunohost Git/Infra notifications> [package_linter] @orhtej2 opened [pull request #118](https://github.com/YunoHost/package_linter/pull/118): Consistently parse apt dependencies.
[20:48:34]
<orhtej2> > <@yunohostinfra:matrix.org> [package_linter] @orhtej2 opened [pull request #118](https://github.com/YunoHost/package_linter/pull/118): Consistently parse apt dependencies.
that should fix it :P
[20:49:59]
<Yunohost Git/Infra notifications> [package_linter] @Tagadda approved [pull request #118](https://github.com/YunoHost/package_linter/pull/118#pullrequestreview-1614194460) Consistently parse apt dependencies.
[20:50:21]
<orhtej2> indeed fixed it in local testing
[20:50:27]
<Yunohost Git/Infra notifications> [package_linter] @Tagadda merged [pull request #118](https://github.com/YunoHost/package_linter/pull/118): Consistently parse apt dependencies.
[20:50:28]
<Yunohost Git/Infra notifications> [package_linter] @Tagadda pushed 2 commits to master ([c6cbd26235b3...b5b40510c973](https://github.com/YunoHost/package_linter/compare/c6cbd26235b3...b5b40510c973))
[20:50:30]
<Yunohost Git/Infra notifications> [package_linter/master] Consistently parse apt dependencies. Make "x y" proper list of dependencies [x, y] rather that of [x y] - orhtej2
[20:50:35]
<Yunohost Git/Infra notifications> [package_linter/master] Merge pull request #118 from orhtej2/patch-1 Consistently parse apt dependencies. - Tagada
[22:23:15]
<selfhoster1312> orhtej2, i don't understand why change the linter to not match yunohost code??
https://github.com/YunoHost/yunohost/blob/dev/src/utils/resources.py#L1094
[22:23:45]
<selfhoster1312> (maybe i missed something)