FlowKit icon indicating copy to clipboard operation
FlowKit copied to clipboard

make file fails

Open bilgecag opened this issue 3 years ago • 4 comments

Describe the bug

Following the instructions at https://flowkit.xyz/developer/dev_environment_setup/ after setting the env vars, make up should start the containers. However, flowdb doesn't build.

Product Makefile

Version aeca40c

To Reproduce In the main FlowKit directory, run

set -a && . development_environment && set +a
make up

Expected behavior

[...] E: Version '3.1.4+dfsg-3.pgdg110+1' for 'postgresql-12-postgis-3' was not found E: Version '3.1.4+dfsg-3.pgdg110+1' for 'postgresql-12-postgis-3-scripts' was not found E: Version '3.1.4+dfsg-3.pgdg110+1' for 'postgis' was not found The command '/bin/sh -c apt-get update && apt-get install -y --no-install-recommends postgresql-$PG_MAJOR-postgis-$POSTGIS_MAJOR=$POSTGIS_VERSION postgresql-$PG_MAJOR-postgis-$POSTGIS_MAJOR-scripts=$POSTGIS_VERSION postgresql-$PG_MAJOR-pgrouting=$PGROUTING_VERSION postgresql-$PG_MAJOR-ogr-fdw=$OGR_FDW_VERSION postgresql-$PG_MAJOR-tds-fdw=$TDS_FDW_VERSION postgresql-server-dev-$PG_MAJOR=$PG_VERSION postgis=$POSTGIS_VERSION && rm -rf /var/lib/apt/lists/* && apt-get purge -y --auto-remove' returned a non-zero code: 100 ERROR: Service 'flowdb' failed to build : Build failed make: *** [Makefile:101: flowdb-build] Error 1

bilgecag avatar Jan 05 '22 14:01 bilgecag

Hi Bilgecag This builds fine on our CI server and my personal machine - do you have the latest version of master?

Thingus avatar Jan 05 '22 16:01 Thingus

Hi @bilgecag. The problem here is that the FlowDB dockerfile is specifying a postGIS version that is no longer available from APT (APT drops old versions very soon after new ones are released).

Looks like you're using the code from FlowKit release 1.16.0 here. This has been fixed in the latest version, so as @Thingus says you should be able to build flowdb without problems if you use the latest version of the master branch. Alternatively, if you want to stick with version 1.16.0 then the pre-built image on DockerHub should work fine (docker pull flowminder/flowdb:1.16.0), but you won't be able to build the image locally.

jc-harrison avatar Jan 05 '22 16:01 jc-harrison

It would probably be worth mentioning this in the docs I think.

greenape avatar Jan 06 '22 09:01 greenape

Thank you for your help, the latest version works fine indeed.

bilgecag avatar Jan 06 '22 12:01 bilgecag