#5 this (hopefully) complete issue 2

Merged
SlavekB merged 1 commits from issue/2 into master 7 months ago
cethyel commented 8 months ago

I didn’t see the need for the libvisual stuff, so I got rid of It.
config.h.cmake has been written as the usual…

I didn't see the need for the libvisual stuff, so I got rid of It. config.h.cmake has been written as the usual...
SlavekB reviewed 8 months ago
It looks good. Just a supplementary question – see below.
@@ -42,4 +43,1 @@

##### optional stuff ############################

# option( WITH_LIBVISUAL "Enable libvisual support" OFF )
SlavekB

Were you considering using the usual options WITH_ALL_OPTIONS and WITH_GCC_VISIBILITY here?

Were you considering using the usual options WITH_ALL_OPTIONS and WITH_GCC_VISIBILITY here?
SlavekB approved these changes 7 months ago
It looks good.
SlavekB deleted branch issue/2 7 months ago
SlavekB added this to the R14.0.6 release milestone 7 months ago

Reviewers

SlavekB approved these changes 7 months ago
The pull request has been merged.
Sign in to join this conversation.
Loading…
Cancel
Save
There is no content yet.