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

Setting secpoll PACKAGEVERSION is broken #2030

Closed
zeha opened this issue Dec 31, 2014 · 2 comments
Closed

Setting secpoll PACKAGEVERSION is broken #2030

zeha opened this issue Dec 31, 2014 · 2 comments

Comments

@zeha
Copy link
Collaborator

zeha commented Dec 31, 2014

The previously supported way of doing

CXXFLAGS += -DPACKAGEVERSION='"$(version)"'

no longer works and fails with:

secpoll-auth.cc: In function 'void doSecPoll(bool)':
<command-line>:0:16: error: invalid operands of types 'const char [6]' and 'const char [26]' to binary 'operator+'
secpoll-auth.cc:127:27: note: in expansion of macro 'PACKAGEVERSION'
   string query = "auth-" +PACKAGEVERSION +".security-status."+::arg()["security-poll-suffix"];
                           ^
<command-line>:0:16: error: invalid operands of types 'const char [48]' and 'const char [26]' to binary 'operator+'
secpoll-auth.cc:154:76: note: in expansion of macro 'PACKAGEVERSION'
     L<<Logger::Warning<<"Could not retrieve security status update for '" +PACKAGEVERSION+ "' on '"+query+"', RCODE = "<< RCode::to_s(res)<<endl;
                                                                            ^

@zeha zeha added the auth label Dec 31, 2014
@Habbie
Copy link
Member

Habbie commented Dec 31, 2014

This was broken in 77b9f5f. Perhaps we just need to string(PACKAGEVERSION).

@Habbie
Copy link
Member

Habbie commented Dec 31, 2014

As @zeha points out on IRC, this should be a configure flag.

@Habbie Habbie added the defect label Dec 31, 2014
@Habbie Habbie closed this as completed in ec1893e Dec 31, 2014
mind04 pushed a commit to mind04/pdns that referenced this issue Apr 30, 2015
mind04 pushed a commit to mind04/pdns that referenced this issue Apr 30, 2015
mind04 pushed a commit to mind04/pdns that referenced this issue Apr 30, 2015
mind04 pushed a commit to mind04/pdns that referenced this issue Apr 30, 2015
mind04 pushed a commit to mind04/pdns that referenced this issue May 1, 2015
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants