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

Staging next #66381

Merged
merged 176 commits into from Aug 18, 2019
Merged

Staging next #66381

merged 176 commits into from Aug 18, 2019

Conversation

FRidh
Copy link
Member

@FRidh FRidh commented Aug 9, 2019

Motivation for this change
Things done
  • Tested using sandboxing (nix.useSandbox on NixOS, or option sandbox in nix.conf on non-NixOS)
  • Built on platform(s)
    • NixOS
    • macOS
    • other Linux distributions
  • Tested via one or more NixOS test(s) if existing and applicable for the change (look inside nixos/tests)
  • Tested compilation of all pkgs that depend on this change using nix-shell -p nix-review --run "nix-review wip"
  • Tested execution of all binary files (usually in ./result/bin/)
  • Determined the impact on package closure size (by running nix path-info -S before and after)
  • Ensured that relevant documentation is up to date
  • Fits CONTRIBUTING.md.
Notify maintainers

cc @

bhipple and others added 30 commits May 24, 2019 17:29
This seems to have recently resolved itself with updates
and is no longer necessary.
Not necessary since #52856

Parallel building is enabled by default with CMake.
dependencies:
perlPackages.AnyURIEscape: init at 0.01
perlPackages.ConfigOnion: init at 1.007
perlPackages.DataUtil: init at 0.66
perlPackages.DispatchClass: init at 0.02
perlPackages.LogAnyAdapterLog4perl: init at 0.09
perlPackages.MooXAliases: init at 0.001006
perlPackages.ParserMGC: init at 0.16
perlPackages.PathIteratorRule: init at 1.014
perlPackages.TestFilename: init at 0.03
perlPackages.TestLWPUserAgent: init at 0.033
perlPackages.TextHogan: init at 2.02
perlPackages.TryTinyByClass: init at 0.01
perlPackages.URITemplate: init at 0.24
dependencies:
perlPackages.TimeOut: init at 0.11
perlPackages.URISmartURI: init at 0.032
@vcunat
Copy link
Member

vcunat commented Aug 10, 2019

@NixOS/darwin-maintainers: there's been a mass failure for about a week (I think) – swift-corefoundation doesn't build (repeatedly), can some of you have a look?

@LnL7
Copy link
Member

LnL7 commented Aug 13, 2019

Unfortunately I can't seem to reproduce that locally. (both on 10.12 and 10.14)

[175/180] CompileC: String.subproj/CFRegularExpression.c
[176/180] CompileC: String.subproj/CFRunArray.c
[177/180] CompileC: URL.subproj/CFURL.c
[178/180] CompileC: String.subproj/CFString.c
[179/180] CompileC: Base.subproj/CFKnownLocations.c
[180/180] Link: Build/CoreFoundation/libCoreFoundation.dylib
ld: warning: directory not found for option '-L../bootstrap/x86_64-apple-darwin/usr/lib'
ld: warning: undefined base symbol '__TMC15SwiftFoundation19_NSCFConstantString' for alias '___CFConstantStringClassReference'
installing
post-installation fixup
strip is /nix/store/68cs638458sifx94qila1xwpbmw88yrq-cctools-binutils-darwin/bin/strip
patching script interpreter paths in /nix/store/0y4lhd3n4dq1v4771p703jcxrbwcg4i1-swift-corefoundation
warning: rewriting hashes in '/nix/store/lpgv4agfpz7p08pnvp63n20k7ffhhznq-swift-corefoundation'; cross fingers
/nix/store/lpgv4agfpz7p08pnvp63n20k7ffhhznq-swift-corefoundation

@grahamc
Copy link
Member

grahamc commented Aug 13, 2019

I just managed to run the build on 10.13.6 which is what hydra runs.

@vcunat
Copy link
Member

vcunat commented Aug 14, 2019

And it succeeded?

Running pass 'Interprocedural Sparse Conditional Constant Propagation' on module 'Base.subproj/CFRuntime.c'.

Perhaps it's possible to work around it by tweaking optimization parameters for this package/file.

@FRidh
Copy link
Member Author

FRidh commented Aug 17, 2019

@NixOS/darwin-maintainers are there any ideas how to proceed?

@FRidh FRidh merged commit 295888c into master Aug 18, 2019
@FRidh FRidh mentioned this pull request Aug 18, 2019
@FRidh
Copy link
Member Author

FRidh commented Aug 18, 2019

opened a separate issue for darwin issue #66811

@grahamc
Copy link
Member

grahamc commented Aug 18, 2019

We were able to reproduce it when running it in a loop... but only sometimes. Ugh.

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.

None yet