* [builds] Improve mono/llvm dependencies.
* Create a list of all the files in the mono and llvm repositories, and save
these lists as a Make variable (in a generated Makefile - .deps.*.mk). We
don't list _all_ the files in each repository, because there are quite a few
(55k for mono), and Make measurably takes a while to check all of them, so
try to limit it to a sane subset, without risking missing changes to files
that actually matters.
* Always create stamp files when we're done with mono builds.
* Modify the mono/llvm builds to depend on all the files in their
repositories.
* Explicitly list the corresponding .stamp-build-* files as dependencies for
various files that are produced by the mono builds, so that make knows how
to build these files.
* Rewrite the *-facade-check targets to depend on the corresponding
*_BCL_TARGETS, so that we can avoid running a submake to the same Makefile
to execute the facade checks.
It now takes a little while (less than a second on my machine, which is
fine) for make to list all dependencies and get their timestamps, but if
executing multiple submakes this adds up to a multi-second timewaste.
So avoid the timewaste by not doing submakes, but instead use dependencies
to enforce the required target execution ordering.
* Don't depend on nicely named intermediate targets, since won't prevent
rebuilds:
build-cross64: setup-cross64
Since the `setup-cross64` file doesn't exist, `build-cross64` will always
execute. Instead depend on the stamp file:
build-cross64: .stamp-configure-cross64
And now `build-cross64` will only rebuild if needed.
* Don't try to list all intermediate files as .SECONDARY dependencies, instead
list none at all, which works as if all files were listed as dependencies.
* Some targets had to move later in the file, since variables used in dependencies:
foo: $(VARIABLE)
must be defined before that point in the file, as opposed to variables used in recipes:
foo:
$(MAKE) $(VARIABLE)
can be defined anywhere in the Makefile.
* Simplify the targets that sign assemblies significantly.
There are a few end results:
* It's now possible to do `make install`, without doing `make all` first. This
might seem weird, but that also ensures the more common `make all install`
works properly.
* Remakes (without any mono/llvm changes) in build/ are much faster, because
we now won't recurse into every mono build:
$ time make all -C builds/ -j8
[...]
real 0m1.873s
This even means that we might be able to make it a habit to remake in the
root directory, which doesn't take forever now:
$ time make all -j8
[...]
real 0m4.521s
Unfortunately adding `make install` to the mix still does some useless
stuff, and it ends up taking ~30 seconds to complete a full build:
$ time make all install -j8
[...]
real 0m32.542s
* [msbuild] Don't verify the xml syntax of targets files unless the files change.
* [build] Don't depend on installed files.
Don't depend on installed files, because that causes a rebuild when installing
to a different directory (i.e. package creation).
* Bump maccore to get build improvements.
Rebuilds are now very fast:
$ make all install -j8
$ time make all install -j8
real 0m5.735s
Less than 6s to figure out that nothing needs to be done.
And strangely flushing the disk cache doesn't make it much slower:
$ sudo purge
$ time make all install -j8
real 0m7.309s
Which probably means that Make mostly reads file metadata, and not actual file
contents (which is good).
This fixes a file sharing exception:
> MTOUCH: error MT1009: Could not copy the assembly '[...]/msbuild/tests/MyActionExtension/bin/iPhone/Debug/MyActionExtension.dll' to '[...]/msbuild/tests/MyTabbedApplication/obj/iPhone/Debug/mtouch-cache/32/Link/MyActionExtension.dll': Sharing violation on path [...]/msbuild/tests/MyActionExtension/bin/iPhone/Debug/MyActionExtension.pdb
- In some build cases this chunk of code:
<ItemGroup Condition=" '$(NoCompilerStandardLib)' == 'true' and '$(NoStdLib)' != 'true' ">
<!-- Note that unlike VB, C# does not automatically locate System.dll as a "standard library"
instead the reference is always passed from the project. Also, for mscorlib.dll
we need to provide the explicit location in order to maintain the correct behaviour
-->
<_ExplicitReference Include="$(FrameworkPathOverride)\mscorlib.dll" />
</ItemGroup>
would trigger and force us to use mscorlib from system mono. That does not work well.
- By setting FrameworkPathOverride, we can get the right mscorlib
- However, that ItemGroup happens outside of a target, so we must move our setting to match for it to take effect
* [tests] Don't create test packages by default.
Don't create test packages by default, instead add a new target to create test
packages. This new target is called on wrench, which means the packages will
still be created when needed, but they won't be built locally in every build
(and if a packaged test fails to build, it won't fail the entire build).
* [tests] Use a project reference instead of assembly reference for GuiUnit.exe
Use a project reference instead of assembly reference for GuiUnit.exe, so that
the GuiUnit reference is automatically built if necessary.
This also makes it required to build a sln for Classic (since mdtool can't
find referenced projects from a csproj).
https://bugzilla.xamarin.com/show_bug.cgi?id=52730
There is no need for bindas support here, by just using bool and
[NullAllowed] the generator does the right thing
```csharp
[CompilerGenerated]
public System.Boolean? WasSuspended {
get {
IntPtr value;
if (k2 == IntPtr.Zero)
k2 = ObjCRuntime.Dlfcn.GetIntPtr (Libraries.AVFoundation.Handle, "AVAudioSessionInterruptionWasSuspendedKey");
if (Notification.UserInfo == null)
return null;
value = Notification.UserInfo.LowlevelObjectForKey (k2);
if (value == IntPtr.Zero)
return null;
using (var nsn = Runtime.GetNSObject<NSNumber> (value))
return nsn.BoolValue;
}
}
```
In a6b9c28975 I fixed SystemSoundTest.FromFile
to not crash randomly, and at the same time I added an assert to ensure that
the playback completion handler is called properly, with a 10s timeout.
Which won't work, because the audio file is 23s long.
There's no need for a 23s audio file, so the fix is simple: cut the audio file
to 0.3s instead.
https://bugzilla.xamarin.com/show_bug.cgi?id=54236
* Bump to mono master
* [build] Profiles cleanup
* Remove 4.x profile dependency
* Replace mdbrebase tool with pathmap and fix broken debug symbols mapping in public builds
* Bump mono and linker submodule
* Add netstandard to the build
* [build] Register linker for submodule checks
* Bump mono
* [build] Add .NET Core open source key
* [build] Update debug symbols to pdb
* [msbuild] Remove smcs compiler overrides
* Update after another Cecil breaking change
* Fixes mmp execution during build to use correct Mono.Cecil. For some reason setting MONO_PATH does not work
* [linker] Update list of Facade assemblies
* [mtouch] Copy pdb debug symbols when available
* [msbuild] Update test to reflect Mono.Dynamic.Interpreter.dll removal
* [tuner] Update System.Core rule
* [msbuild] Update few test expected debug symbols
* Fixes missing Mono.Security dependency
* Updates mtouch.csproj
* Switch to Mono 2017-02 branch
* Adds pdb handling to mtouch
* Bump linker
* Wire up configuration to MonoTouchSweepStep
* Mono branch name update
* Remove mdbrebase which is no longer needed (compiler does it now)
* Update monomac tuner
* Revert "[linker] Inline BinaryCompatibility.TargetsAtLeast_Desktop_V4_5[_1] (#1518)"
This reverts commit e72b654890.
because it's no longer needed with latest mono
* Bump maccore for mdoc path fix
* Add missing reference
* Bump linker
* Updates mmp.csproj files path
* [tests][mac] Terminate test processes with extreme prejudice. Fixes#52604.
https://bugzilla.xamarin.com/show_bug.cgi?id=52604
* Bump Xamarin Studio to get fix for bug #52648.
Bump Xamarin Studio to get fix for bug #52648 causing build failures.
* Update system Mono to 2017-02
* More mdb to pdb checks update
* Update tests to work with csc compiler
* Another system mono bump
* [tests] Fix mlaunch path, it's no longer shipped with Xamarin Studio. (#1842)
* Fixes tuner IL optimization for get_Size to work with short branches
* Update IntPtrSizeOptimization test to consider Debug mode IL output
* Xamarin.Mac.FSharp.targets - Fix imports to work correctly with ..
.. MSBuildExtensionsPath* .
`MSBuildExtensionsPath*` supports search paths only if the string
`$(MSBuildExtensionsPath)` (or *32/*64) appear in the `Project`
attribute of the `Import` element. Otherwise the value of the property
or the default value is used. So, restructure to conditionally import
rather than a import using a property with the final path.
* [msbuild] Index into the correct item array when printing an error message (#1792)
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=52860
When the ImageAssets contain an item that doesn't live within
a *.xcassets directory, index into ImageAssets[] rather than
items[] which hasn't been populated yet.
Also fixed the tagsList for-loop to use tagsList.Count instead
for correctness (even though tags.Count and tagsList.Count
should always be identical).
* [msbuild] Ignore .DS_Store files when cloning asset catalogs (#1776)
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=52851
The problem here is that the .DS_Store file was included in
the .csproj file but did not exist on disk, so when we went
to clone that file into the obj/ dir before running actool
on it, File.Copy() would fail because the file did not
actually exist.
Since these files are worthless anyway, we can safely ignore
them.
Also added logic to verify that files exist before copying
them in order to report a better error than an exception
stack trace.
* [msbuild] Make a build error more clearly worded (#1794)
Related to https://bugzilla.xamarin.com/show_bug.cgi?id=52847
* [msbuild] Remove an unused enum that causes confusion (#1798)
* [msbuild] sanity check TargetiOSDevice property for conflicts (#1799)
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=52847
If the device archs do not match the configuration, don't
do a device-specific build.
* [msbuild] Make sure to codesign appex dylibs (#1812)
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=52745
Added a new CodesignNativeLibraries task that scans for
and then codesigns each *.dylib and *.metallib in the
app bundle (minus those in the PlugIns and Watch dirs).
* [msbuild] Run the _ComputeTargetArchitectures target before cleaning (#1828)
Some of the Clean steps need the _ComputeTargetArchitectures
target to run before they can properly do their thing because
they depend on DeviceSpecific paths.
Also added logic to rm -rf extension *.dSYM and framework *.dSYM
dirs in the container app bin dir (which fails on xbuild due to
a bug in xbuild... yay!)
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=53007
* [msbuild] ALWAYS log ibtool/actool exitCode != 0 as an error (#1840)
Log this as an error even if the log file does not exist and/or
is parseable.
Previously, it was possible that if the log file didn't exist
or it was perfectly validly formatted, we would never log that
error code and so it's possible that we wouldn't log *any*
error at all (e.g. empty log file).
* [msbuild] Added logging for all of the overridden MSBuild default tasks (#1829)
* [msbuild] Fixed PathUtils.AbsoluteToRelative() logic (#1861)
* [msbuild] Fixed PathUtils.AbsoluteToRelative() logic
Don't require the 'absolute' path argument to be a FullPath.
The code already calls Path.GetFullPath() on the 'absolute'
path anyway.
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=53176
* Fixed ResolveSymbolicLinks() to return the input path if realpath() fails
* [msbuild] Use stamp files to force container app's _CompileToNative (#1878)
Fixes https://bugzilla.xamarin.com/show_bug.cgi?id=53303
* Bump mono
* Another mono bump
* Mono bump
* Update .gitignore with pdb files
* Bump mono
* Mono bump
* Revert "Bump maccore for mdoc path fix"
This reverts commit 50f4564b6b.
* Bump maccore
* Bump mono
* Remove TlsProviderFactoryCallback
* Update linker tests after appletls merge
* Mono bump
* Bump mono to include fix for bug #53481
* Bump mono
* Bump system mono to get fix for #53481.
https://bugzilla.xamarin.com/show_bug.cgi?id=53481
* Bump mono
* Bump mono
* Remove Mono.Dynamic.Interpreter from mtouch tests
Type Changed: ModelIO.MDLMesh
Removed method:
public static MDLMesh CreateBox (OpenTK.Vector3 dimensions, OpenTK.Vector3i segments, MDLGeometryType geometryType, bool inwardNormals, IMDLMeshBufferAllocator allocator);
---
Adding an optional argument (so not having the original signature anymore) would break binary compatibility.
Add an assembly registration event, that allows apps to opt out of
Xamarin.Mac's default behavior to recursively load every assembly referenced
by the entry assembly.
This is only for Xamarin.Mac, since it does not make sense to have this API in
Xamarin.iOS because assemblies are statically registered at build time.
In Xcode8.3 branch I [introduced a breaking change](0) by adding
[DisableDefaultCtor] to `INPreferences`. While this change is
correct we need it to happen until XAMCORE_4_0 becomes a thing.
Fixes
```
Type Changed: Intents.INPreferences
Removed constructor:
public INPreferences ();
```
[0]: 5511768914 (diff-efb680ffd3a9a21fd9c1c83fe8be84bfR2738)
A new method added in Xcode 8.3 triggeres a new warning in d15-1 about
exposing models instead of protocols in API.
Bind this method correctly (using a protocol), and add a compatibility
version.
Fixes this introspection error:
[FAIL] `MDLAsset.PlaceLightProbes` includes a parameter of type `MDLLightProbeIrradianceDataSource` which is a concrete type `[Model]` and not an interface `[Protocol]`
[FAIL] iOSApiSignatureTest.ApiSignatureTest.ManagedSignature : 1 errors found in 3 signatures validated:
Apple added API_UNAVAILABLE(watchos) to
+ (INSiriAuthorizationStatus)siriAuthorizationStatus
+ (void)requestSiriAuthorization:(void (^)(INSiriAuthorizationStatus status))handler
So we got the availability right :D
Fixes bug #52575: Missing ModelIO API
(https://bugzilla.xamarin.com/show_bug.cgi?id=52575)
We turn the MDLMesh constructors into static ones because we don't want to lose the shape name.
Also, the signatures of these constructors differ so it would not be possible to use an enum to differentiate the shapes.
watchOS 3.2 introduced SiriKit extension that must be bundled
inside the Watch App Extension (yep a extension inside another extension).
So the _ResolveAppExtensionReferences and _CopyAppExtensionsToBundle should
now be run on extension projects too.
This should stop errors like below to happen on wrench
```
Xamarin.Bundler.MonoMacException: The type 'AVFoundation.IAVContentKeyRecipient' (used as a parameter in AVFoundation.AVContentKeySession.Add) is not available in macOS 10.12.2 (it was introduced in macOS 10.12.4). Please build with a newer macOS SDK (usually done by using the most recent version of Xcode).
```
The return type is wrong
> Value is a boolean NSNumber,
and cannot be bound as a `bool` but as a `NSNumber`, which makes it
closer to a `bool?` since it could be absent.
We'll re-enable it when merging `xcode8.3` on `master` so we can benefit
from Alex `[BindAs]` support, giving us a `bool?` on the visible API
Mac introspection does not like it on wrench, which seems to
have a Sierra beta (or it would be ignored)
> [FAIL] Selector not found for Intents.INPerson : siriMatches
and xtro / headers can't see it being available on macOS either
Sebastien Pouliot
Rolf Bjarne Kvinge
Chris Hamons
Alex Soto
Håvard Fossli
Vincent Dondain
Marek Safar
Timothy Risi
Manuel de la Pena