Skip to content

Remove pgsql marker from a couple SNPRC modules#897

Merged
labkey-adam merged 1 commit intodevelopfrom
fb_no_pgsql_for_now
Feb 26, 2026
Merged

Remove pgsql marker from a couple SNPRC modules#897
labkey-adam merged 1 commit intodevelopfrom
fb_no_pgsql_for_now

Conversation

@labkey-adam
Copy link
Contributor

Rationale

I marked snprc_ehr and snprc_genetics as supporting PostgreSQL during my initial migration work. This caused them to end up in the all_pg distribution (because all PostgreSQL-supporting modules get pulled in), but they also brought in their snd dependency, which does not yet support PostgreSQL. That causes problems when trying to run this distribution in production mode. Best to remove the pgsql tag for these modules until they truly support PostgreSQL.

I left snprc_r24 tagged with pgsql because it has no dependencies and should run fine on PostgreSQL.

Related Pull Requests

@labkey-adam
Copy link
Contributor Author

@labkey-adam labkey-adam merged commit 0638776 into develop Feb 26, 2026
7 of 9 checks passed
@labkey-adam labkey-adam deleted the fb_no_pgsql_for_now branch February 26, 2026 04:37
@labkey-adam
Copy link
Contributor Author

Doing a custom all_pg installers build here: https://teamcity.labkey.org/buildConfiguration/LabKey_Trunk_Premium_InstallersCustom/3866421

Looks good... no snd. snprc_r24 is the only snprc module.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants