OculusManifestPreprocessor creates broken AndroidManifest for Quest

[Dino Fejzagić]

XRTK - Mixed Reality Toolkit Bug Report

Describe the bug

The OculusManifestPreprocessor generated a broken AndroidManifest.xml when used via MRT -> Tools -> Oculus -> Create Ouclus Quest compatible AndroidManifest.xml

It generates a manifest with a duplicate entry for hand tracking:

<manifest xmlns:android="http://schemas.android.com/apk/res/android"
    android:installLocation="auto">
  <uses-feature android:name="oculus.software.handtracking" android:version="1" android:required="true" /><!-- Request the headset DoF mode -->
  <uses-feature android:name="oculus.software.handtracking" android:required="false" /><uses-permission android:name="com.oculus.permission.HAND_TRACKING" /><!-- Request the headset handtracking mode -->
  <application...

Note how there is two lines with the same content except one says android:required="true" and the other one android:required="false".

To Reproduce

Remove any existing AndroidManifest.xml and regenerate it using the manifest menu item.

Expected behavior

The hand tracking entry is only added once.

Actual behavior

The hand tracking entry is added twice.