Looks good to me. I tested it with this in config.xml:
<plugin name="cordova-plugin-geolocation" source="npm"
spec="https://github.com/apache/cordova-plugin-geolocation.git#4.1.0";>
<variable name="GPS_REQUIRED" value="false" />
</plugin>
After cordova platform add, this appears in AndroidManifest.xml"
<uses-feature android:name="android.hardware.location.gps"
android:required="false" />
Thumbs up!
- George Henne
