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

Application asks "Do you want to quit ? [Y]/n" when piping into the repl #13

Open
fstamour opened this issue Dec 23, 2022 · 5 comments
Open
Assignees

Comments

@fstamour
Copy link

For testing an unnamed book application I wrote something like this:

#!/usr/bin/env bash

cat <<EOF |
locale
stock
locale fr_FR
stock
EOF
./bookshops -i

Which resulted in:

image

@vindarel
Copy link
Owner

ACK. I'll need to give this a look, in a few days.

an unnamed book application

🤣

(ps: happy family moments 🎄 🎁 )

@vindarel
Copy link
Owner

Supposedly we have:

;; config.lisp in openbookstore's root directory
(setf replic:*confirm-exit* nil)

Or in ~/.config/.replic.conf

[default]
confirm-exit = false

but I can't see their effect.

@vindarel vindarel self-assigned this Dec 25, 2022
@vindarel
Copy link
Owner

vindarel commented Jan 4, 2023

The configuration setting works fine for the replic app itself. This is an issue with OpenBookStore which doesn't read it.

I'll keep the issue here, replic should probably read its config file when used from another app.

vindarel added a commit that referenced this issue Jan 4, 2023
previously, "section" and "package" were the same, we could not read
the ini config file for another app.

Now, simply do

(replic.config:apply-config)

to load replic's parameters from the default section.

Works for confirm-exit.

for #13
@vindarel
Copy link
Owner

vindarel commented Jan 4, 2023

I pushed a fix to replic and to OpenBookStore.

Try with this:

# ~/.replic.conf
[default]
 confirm-exit = false

@vindarel
Copy link
Owner

vindarel commented Jan 4, 2023

bummer. You also have the quit command which does not ask for confirmation.

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

No branches or pull requests

2 participants