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

lpadmin command fails when using CUPS_LPADMIN_PRINTER env variable #93

Open
cruzrealestate opened this issue Jun 14, 2024 · 3 comments
Open
Labels
question Further information is requested

Comments

@cruzrealestate
Copy link

cruzrealestate commented Jun 14, 2024

first of all, great job on this container. does everything I need except, line 102 in your start-cups.sh..

when i run my lpadmin command via an interactive session, my printer gets created. when i try to specify it via the yaml file, i get the message that the file cannot be found..

@DrPsychick
Copy link
Collaborator

Can you share the output of your container? It should print out the values it extracted from the ENV. (line 102).
Also, if your lpadmin command relies on an additional file, you will probably need to mount it into the container.

@silentkiller88
Copy link

I had the same issue if I put the command between double quotes in my docker compose file
It worked fine after removing the quotes.

CUPS_LPADMIN_PRINTER1 = "lpadmin -p Brother_HL2150N -D Brother_HL-2150N_series -P /etc/cups/ppd/Brother_HL-2150N_series.ppd -v lpd://192.168.1.8/BINARY_P1"
/root/start-cups.sh: line 103: lpadmin -p Brother_HL2150N -D Brother_HL-2150N_series -P /etc/cups/ppd/Brother_HL-2150N_series.ppd -v lpd://192.168.1.8/BINARY_P1: No such file or directory

@DrPsychick
Copy link
Collaborator

@silentkiller88 you ran into the issue that docker ENV variables are quite limited (see https://github.com/SickHub/docker-cups-airprint?tab=readme-ov-file#variables-overview)

No quotes, no newlines - not what one would typically expect from environment variables.

@DrPsychick DrPsychick added the question Further information is requested label Sep 5, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
question Further information is requested
Projects
None yet
Development

No branches or pull requests

3 participants