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

build failure vs cups 2.3.0 #11

Closed
dagbdagb opened this issue Sep 29, 2019 · 3 comments
Closed

build failure vs cups 2.3.0 #11

dagbdagb opened this issue Sep 29, 2019 · 3 comments

Comments

@dagbdagb
Copy link

make bombs at:

depbase=echo ../common/CupsPrintEnvironment.o | sed 's|[^/]*$|.deps/&|;s|\.o$||';
g++ -DHAVE_CONFIG_H -I. -I../../src -I../common -O2 -Wall -Wno-unknown-pragmas -MT ../common/CupsPrintEnvironment.o -MD -MP -MF $depbase.Tpo -c -o ../common/CupsPrintEnvironment.o ../common/CupsPrintEnvironment.cpp &&
mv -f $depbase.Tpo $depbase.Po
../common/CupsPrintEnvironment.cpp: In member function ‘virtual void DymoPrinterDriver::CCupsPrintEnvironmentForLM::ReadData(DymoPrinterDriver::buffer_t&)’:
../common/CupsPrintEnvironment.cpp:139:23: error: ‘cupsBackChannelRead’ was not declared in this scope
ssize_t bytesRead = cupsBackChannelRead((char*)buf, sizeof(buf), 2.5);
^~~~~~~~~~~~~~~~~~~

Also see:
https://bugs.gentoo.org/693652

Also a fair number of warnings about deprecated functions when built with -Wdeprecated-declarations .

@matthiasbock
Copy link
Owner

Thank you for your report. Unfortunately, I can't reproduce your problem. Also, it seems to me that other sources than mine were used in the linked bug report, so it's of little use to me. If you believe, the problem there also applies to this repository, please create a fresh clone of this repository and provide the full console output of all your compilation steps. Please also tell me your compiler versions. Thank you.

@jkl1337
Copy link

jkl1337 commented Oct 27, 2019

This repo is indeed affected when trying to build against cups 2.3.0. cups/sidechannel.h must be added explicility to CupsPrintEnvironment.cpp.

There are also numerous warnings about ppdClose deprecation and similar. For now that is not breaking build

@matthiasbock
Copy link
Owner

Thank you, @jkl1337. Also to you: Please create a fresh clone of this repository and provide the full console output of your compilation process.

If you have a patch, please upload it here (the patch or the patched file) or kindly make a pull request.

codeprofis added a commit to codeprofis/dymo-cups-drivers that referenced this issue Mar 12, 2020
Add missing backend header. Fixes matthiasbock#11.
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

3 participants