Skip to content

fix: Fix compilation on Windows with spaces in the Rtools path#1000

Merged
aviator-app[bot] merged 3 commits intomainfrom
ucrt-include
Dec 14, 2023
Merged

fix: Fix compilation on Windows with spaces in the Rtools path#1000
aviator-app[bot] merged 3 commits intomainfrom
ucrt-include

Conversation

@Antonov548
Copy link
Copy Markdown
Contributor

@Antonov548 Antonov548 commented Dec 14, 2023

Closes #992.

@aviator-app
Copy link
Copy Markdown
Contributor

aviator-app bot commented Dec 14, 2023

Current Aviator status

Aviator will automatically update this comment as the status of the PR changes.
Comment /aviator refresh to force Aviator to re-examine your PR (or learn about other /aviator commands).

This PR was merged using Aviator.


See the real-time status of this PR on the Aviator webapp.

Use the Aviator Chrome Extension to see the status of your PR within GitHub.

Copy link
Copy Markdown
Contributor

@krlmlr krlmlr left a comment

Choose a reason for hiding this comment

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

Looks good. Should we also tweak Makevars and perhaps other files while we're at it?

@krlmlr krlmlr changed the title fix: adding quotation for xml includes fix: Fix compilation on Windows with spaces in the Rtools path Dec 14, 2023
@krlmlr
Copy link
Copy Markdown
Contributor

krlmlr commented Dec 14, 2023

Thanks!

@aviator-app aviator-app bot merged commit d633a20 into main Dec 14, 2023
@aviator-app aviator-app bot deleted the ucrt-include branch December 14, 2023 14:24
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Dec 14, 2024
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Install of igraph 1.6.0 fails under R 4.2.1

2 participants