Skip to content
New issue

Have a question about this project? # for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “#”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? # to your account

Fix list.json #9

Merged
merged 2 commits into from
Aug 4, 2024
Merged

Fix list.json #9

merged 2 commits into from
Aug 4, 2024

Conversation

xtexx
Copy link
Member

@xtexx xtexx commented Aug 4, 2024

  • Drop the unused SQLite dependency
  • Replace group_concat (SQLite) with array_to_string + array_agg DISTINCT (PostgresQL)

not tested

xtexx added 2 commits August 4, 2024 10:15
They are never used.
error returned from database: function group_concat(text) does not exist

group_concat is for sqlite, not postgresql.
@xtexx xtexx requested a review from jiegec August 4, 2024 02:21
xtexx added a commit to xtexx/repology-updater that referenced this pull request Aug 4, 2024
This reverts commit 024146e.

The issue should be fixed by
AOSC-Dev/packages-site-rs#9.
@jiegec jiegec merged commit 9dcd38c into master Aug 4, 2024
1 check passed
@jiegec jiegec deleted the fix-list-json branch August 4, 2024 03:20
xtexx added a commit to xtexx/repology-updater that referenced this pull request Aug 6, 2024
This reverts commit 024146e.

The issue should be fixed by
AOSC-Dev/packages-site-rs#9.
xtexx added a commit to xtexx/repology-updater that referenced this pull request Aug 10, 2024
This reverts commit 024146e.

The issue should be fixed by
AOSC-Dev/packages-site-rs#9.
AMDmi3 pushed a commit to repology/repology-updater that referenced this pull request Aug 12, 2024
This reverts commit 024146e.

The issue should be fixed by
AOSC-Dev/packages-site-rs#9.
# for free to join this conversation on GitHub. Already have an account? # to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants