airsdk / apm

AIR Package Manager
https://repository.airsdk.dev/
MIT License
61 stars 10 forks source link

Update for Android-specific project set `bundleSeedId` value to "" and display error on descriptor generation #192

Open sjabberwocky opened 1 year ago

sjabberwocky commented 1 year ago

Updates for the Android-only variant of ANEs will drop the iOS-specific bundleSeedId value to "" and then display an error message that this value is missing and not generate a descriptor after this.

Thoughts about it:

  1. It does not drop the set value to "".
  2. It should generate descriptor without ios-specific parameters.
marchbold commented 1 year ago

Can you elaborate on what you did to get it to drop the config value?

crooksy88 commented 3 months ago

I believe I've just stumbled upon this.

I am using APM and setting up an Android only app and have:

returns: generate/app-descriptor :: Parameter not valid: bundleSeedId=

Do I need to set bundleSeedId manually to something?

marchbold commented 3 months ago

@crooksy88 If you run apm project config set it will walk through all the config variables for the packages you have installed.

crooksy88 commented 3 months ago

Thanks Michael, my issue though is that this is an Android only app, whereas the bundleSeedId appears to be an Apple requirement.

So without a bundleSeedId from Apple, what should I set this to, as it is a requirement?

I have currently set it to '1' which has removed the error.

Thanks.

marchbold commented 3 months ago

Ah yeah, if you aren't producing an iOS build just set it to any value is fine. I have a bit of work to do around specifying project platform outputs and handling this but I'm working on it.