Skip to content
New issue

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

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

Already on GitHub? Sign in to your account

Observation: unrecognized value "on;" for "ON_ERROR_STOP": Boolean expected #8

Open
dancesWithCycles opened this issue Feb 12, 2024 · 1 comment
Labels
bug Something isn't working help wanted Extra attention is needed

Comments

@dancesWithCycles
Copy link
Contributor

Hi folks,
Have you ever noticed the unrecognized value "on;" for "ON_ERROR_STOP": Boolean expected feedback when calling npm run build? Here is a larger snippet.

...
ALTER TABLE
COPY 53855
COMMIT
+++ realpath ./build.sh
++ dirname /home/pacs/sib00/users/gtfsr_srv_vbb/git/berlin-gtfs-rt-server/build.sh
+ lib=/home/pacs/sib00/users/gtfsr_srv_vbb/git/berlin-gtfs-rt-server/lib
+ sponge
+ psql -b
+ NODE_ENV=production
+ node_modules/.bin/build-gtfs-match-index /home/pacs/sib00/users/gtfsr_srv_vbb/git/berlin-gtfs-rt-server/lib/hafas-info.js /home/pacs/sib00/users/gtfsr_srv_vbb/git/berlin-gtfs-rt-server/lib/gtfs-info.js
stops
routes
trip headsigns
unrecognized value "on;" for "ON_ERROR_STOP": Boolean expected
BEGIN
CREATE TABLE

COPY 579515
CREATE INDEX
...

Cheers!

@derhuerst
Copy link
Owner

derhuerst commented Feb 13, 2024

Have you ever noticed the unrecognized value "on;" for "ON_ERROR_STOP": Boolean expected feedback […]?

Yes, I have! I assume it is caused by this line in gtfs-via-postgres (a dependency of this project):

https://github.com/public-transport/gtfs-via-postgres/blob/4.9.0/index.js#L156

I'd be happy to get it to disappear, especially since the message seems to indicate that the ON_ERROR_STOP setting is not getting activated. 🤔

But I can't reproduce it locally using psql from the Homebrew PostgreSQL v14 package.

@derhuerst derhuerst added bug Something isn't working help wanted Extra attention is needed labels Feb 13, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working help wanted Extra attention is needed
Projects
None yet
Development

No branches or pull requests

2 participants