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

2.1.2 fails to build on Linux #6305

Closed
david-geiger opened this issue Jun 23, 2020 · 4 comments
Closed

2.1.2 fails to build on Linux #6305

david-geiger opened this issue Jun 23, 2020 · 4 comments

Comments

@david-geiger
Copy link

Hi,

I tried on Mageia Cauldron to build latest 2.1.2 release but it fails to build with:

/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build/winpr/include/winpr/version.h:30:22: warning: missing terminating " character
30 | #define GIT_REVISION "[RUN] echo 2.1.2
| ^
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build/winpr/include/winpr/version.h:31:1: error: too many decimal points in number
31 | 2.1.2"
| ^~~~~
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build/winpr/include/winpr/version.h:31:1: error: expected identifier or '(' before numeric constant
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build/winpr/include/winpr/version.h:31:6: warning: missing terminating " character
31 | 2.1.2"
| ^
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build/winpr/include/winpr/version.h:31:6: error: missing terminating " character
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/winpr/libwinpr/utils/winpr.c: In function 'winpr_get_build_revision':
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build/winpr/include/winpr/version.h:30:22: error: missing terminating " character
30 | #define GIT_REVISION "[RUN] echo 2.1.2
| ^~~~~~~~~~~~~~~~~
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/winpr/libwinpr/utils/winpr.c:63:9: note: in expansion of macro 'GIT_REVISION'
63 | return GIT_REVISION;
| ^~~~~~~~~~~~
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/winpr/libwinpr/utils/winpr.c:63:2: warning: 'return' with no value, in function returning non-void [-Wreturn-type]
63 | return GIT_REVISION;
| ^~~~~~
/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/winpr/libwinpr/utils/winpr.c:61:13: note: declared here
61 | const char* winpr_get_build_revision(void)
| ^~~~~~~~~~~~~~~~~~~~~~~~
make[2]: *** [winpr/libwinpr/CMakeFiles/winpr.dir/build.make:762: winpr/libwinpr/CMakeFiles/winpr.dir/utils/winpr.c.o] Error 1
make[2]: Leaving directory '/home/iurt/rpmbuild/BUILD/freerdp-2.1.2/build'
make[2]: *** Waiting for unfinished jobs....

Regards,
David

@akallabeth akallabeth added this to the next milestone Jun 23, 2020
@akallabeth
Copy link
Member

Already fixed with #6301
For the time being remove the additional strings from source_version, sorry for the inconvenience

pld-gitsync pushed a commit to pld-linux/freerdp2 that referenced this issue Jun 23, 2020
- patch for broken release, to be removed in next update
  see FreeRDP/FreeRDP#6305
@bmiklautz
Copy link
Member

The release archives were re-uploaded to fix this issue. Please download them again.

@david-geiger
Copy link
Author

Yes it is fixed now, thank you :)

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

4 participants
@bmiklautz @akallabeth @david-geiger and others