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

[libraw] Add include for select_library_configurations (#6715) #6742

Conversation

trylle
Copy link
Contributor

@trylle trylle commented Jun 3, 2019

Please check if this resolves your problem, @mloskot.

@@ -32,6 +32,8 @@
+ ${PC_LIBRAW_R_LIBRARY_DIRS}
+ )
+
+INCLUDE(SelectLibraryConfigurations)
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Shouldn't this line be placed before the line 21?
i.e. before select_library_configurations(LibRaw)

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Sorry, should be fixed now.

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks good. Thank you!

@trylle trylle force-pushed the libraw_add_missing_select_library_configurations_include branch from 93af2d1 to d2bbf25 Compare June 3, 2019 20:12
@NancyLi1013
Copy link
Contributor

Hi @trylle, thanks for your PR. Could you also bump the version in CONTROL file?

@@ -1,4 +1,4 @@
Source: libraw
Version: 0.19.2
Version: 0.19.2-1
Copy link
Contributor

@mloskot mloskot Jun 6, 2019

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

@trylle This change is no longer necessary due to switch the the LibRaw snapshot (yes, the way they call/manage it may be confusing) in #6759

Copy link
Contributor Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

OK, I've rebased the PR onto master and bumped the version to 201903-1 to differentiate from #6759 already merged into master.

@trylle trylle force-pushed the libraw_add_missing_select_library_configurations_include branch from e71ef11 to d66183d Compare June 6, 2019 16:10
@mloskot
Copy link
Contributor

mloskot commented Jun 6, 2019

Thanks.
What's stopping this from being merged?

@ras0219-msft
Copy link
Contributor

Nothing, thanks for the PR!

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

Successfully merging this pull request may close these issues.

6 participants