Skip to content

Prepare our git-p4 tests for running on APFS #179

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

Closed
wants to merge 2 commits into from

Conversation

dscho
Copy link
Member

@dscho dscho commented Apr 11, 2019

Azure Pipelines' macOS agents were recently upgraded to Mojave, and since that does not support HFS+ anymore, everything will be upgraded to APFS.

As I found the hard way, we have one test that fails on that filesystem (t9822, which is only run if Perforce's p4d is available).

The first patch adds a workaround, as I failed to find any way to finagle APFS into accepting that ISO-8859-1 encoded file name.

The second patch is a fix to make things compile on Mojave again, apparently Homebrew changed and the /usr/local/ directories need to be added to the compiler and linker flags explicitly. Maybe this is a gcc-8 only thing, as it seems to have affected only the osx-gcc job, not the osx-clang job.

Changes since v1:

  • Fix the tyop in the oneline of the first patch where it should talk about ISO-8859-1.

Cc: Luke Diamand [email protected]

@dscho dscho force-pushed the skip-t9822-on-apfs branch from dd73d9c to 52681ae Compare April 12, 2019 09:12
@dscho
Copy link
Member Author

dscho commented Apr 12, 2019

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 12, 2019

Submitted as [email protected]

dscho added 2 commits April 14, 2019 21:34
Most notably, it seems that macOS' APFS does not allow that.

Signed-off-by: Johannes Schindelin <[email protected]>
Due to reasons (some XCode versions seem to include gettext, some
don't?), Homebrew does not expose the libraries and headers in
/usr/local/ by default anymore.

Let's help find them again.

Note: for some reason, this is a change of behavior caused by the
upgrade to Mojave, identified in our Azure Pipeline; it seems that
Homebrew used to add the /usr/local/ directories to the include and link
search path before, but now it no longer does.

Signed-off-by: Johannes Schindelin <[email protected]>
@dscho dscho force-pushed the skip-t9822-on-apfs branch from f006524 to 25be67b Compare April 14, 2019 19:36
@dscho
Copy link
Member Author

dscho commented Apr 14, 2019

/submit

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 14, 2019

Submitted as [email protected]

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 15, 2019

This branch is now known as js/macos-gettext-build.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 15, 2019

This patch series was integrated into pu via git@73a9270.

@gitgitgadget gitgitgadget bot added the pu label Apr 15, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Apr 16, 2019

This patch series was integrated into pu via git@054b28d.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 16, 2019

This patch series was integrated into pu via git@f7944e3.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 18, 2019

This patch series was integrated into pu via git@098f0aa.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 19, 2019

This patch series was integrated into pu via git@11071d0.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 21, 2019

This patch series was integrated into pu via git@f10377b.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 22, 2019

This patch series was integrated into pu via git@32b32c1.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 22, 2019

This patch series was integrated into next via git@de4cbb1.

@gitgitgadget gitgitgadget bot added the next label Apr 22, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Apr 22, 2019

This patch series was integrated into pu via git@27a8f85.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 25, 2019

This patch series was integrated into pu via git@aa8c8d9.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 25, 2019

This patch series was integrated into next via git@aa8c8d9.

@gitgitgadget
Copy link

gitgitgadget bot commented Apr 25, 2019

This patch series was integrated into master via git@aa8c8d9.

@gitgitgadget gitgitgadget bot added the master label Apr 25, 2019
@gitgitgadget gitgitgadget bot closed this Apr 25, 2019
@gitgitgadget
Copy link

gitgitgadget bot commented Apr 25, 2019

Closed via aa8c8d9.

@dscho dscho deleted the skip-t9822-on-apfs branch April 29, 2019 21:59
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

1 participant