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

UPM Package without ExternalDependencyManager #111

Open
snorrsi opened this issue Aug 23, 2022 · 16 comments
Open

UPM Package without ExternalDependencyManager #111

snorrsi opened this issue Aug 23, 2022 · 16 comments

Comments

@snorrsi
Copy link

snorrsi commented Aug 23, 2022

Summary

I noticed that you added ExternalDependencyManager into your tagged versions.
I suppose that is because Google still hasn't bothered to create a Github package download for that.
However, I suppose most of us that are using your package are in one way or another including ExternalDependencyManager in our projects. Which means that when including your package through github url (or other means) will lead to conflict with guids.

To solve this, Can you please create a separate tags for your plugin that have ExternalDependencyManager not added ?

Motivations

Creating separate tags that does not include ExternalDependencyManager solves guid conflicts for those already including it.

@snorrsi
Copy link
Author

snorrsi commented Aug 23, 2022

Hey @dnshi as you added this maybe you comment on this issue of mine :)

@JonLandmark
Copy link

I would also like to request that ExternalDependencyManager not be included in this package. I already have the OpenUPM ExternalDependencyManager as a package in my project so the latest Amplitude SDK will conflict if I update it. Also, conflicts will occur if other SDKs have their own version of the ExternalDependencyManager.

@magicfinger96
Copy link

sorry ive created the same issue earlier. I would like you to remove the EDM4U from your package too. When are you going to do this? It's been almost a month.

@qingzhuozhen
Copy link
Contributor

In v2.6.0, we released amplitude-unity.unitypackage without ExternalDependencyManager; and amplitude-unity-all.unitypackage with ExternalDependencyManager in case any one need it. Please let us know if this helps or any other suggestions. Thanks for raising the concern.

@snorrsi
Copy link
Author

snorrsi commented Sep 20, 2022

In v2.6.0, we released amplitude-unity.unitypackage without ExternalDependencyManager; and amplitude-unity-all.unitypackage with ExternalDependencyManager in case any one need it. Please let us know if this helps or any other suggestions. Thanks for raising the concern.

Doesn´t help for us that use UPM.
Both cases are unresolved there as both ExternalDepency folder and Plugins/Android folder are still there.
It would help if you could create UPM special tag where these folders and meta files are removed and perhaps even only including the conent under /Assets folder to skip having to specify path in UPM

@qingzhuozhen
Copy link
Contributor

@snorrsi Thanks for the feedback. We will work on supporting this.

@snorrsi
Copy link
Author

snorrsi commented Sep 23, 2022

@qingzhuozhen thanks .. I also noticed that there is unneeded Plugins/AmplitudeSDKPlugins.asmdef.meta at https://github.com/amplitude/unity-plugin/blob/main/Assets/Plugins/AmplitudeSDKPlugins.asmdef.meta

It can safely be deleted as Plugins/AmplitudeSDKPlugins.asmdef does not exist (and is not needed as there is no .cs files in the folder or any of it's subfolder) and Unity dislikes meta files without corresponding file

@qingzhuozhen
Copy link
Contributor

Hi @snorrsi, thanks for raising this up. I cut a tag v2.6.0-no-edm without ExternalDependencyManager and the files you mentioned. Please let us know if that helps.

@shamruk
Copy link

shamruk commented Feb 6, 2023

Could you give a UPM link for v2.6.0-no-edm
Like https://github.com/amplitude/unity-plugin?path=/Assets but for v2.6.0-no-edm

Or through openupm
https://openupm.com/packages/com.amplitude.unityplugin/?subPage=pipelines

@snorrsi
Copy link
Author

snorrsi commented Feb 6, 2023

Could you give a UPM link for v2.6.0-no-edm Like https://github.com/amplitude/unity-plugin?path=/Assets but for v2.6.0-no-edm

Or through openupm https://openupm.com/packages/com.amplitude.unityplugin/?subPage=pipelines

@shamruk that link would be

https://github.com/amplitude/unity-plugin.git?path=/Assets#v2.6.0-no-edm

or for the other branch without dependencies

https://github.com/amplitude/unity-plugin.git?path=/Assets#v2.6.0-no-edm-and-dep

have tested neither though as still using my own version

@shamruk
Copy link

shamruk commented Feb 6, 2023

@snorrsi
https://github.com/amplitude/unity-plugin.git?path=/Assets#v2.6.0-no-edm
opens main branch with edm

i'll go with creating own fork for now (just merged tag into main there)
https://github.com/shamruk/unity-plugin.git?path=/Assets
PS also restored asmdef files

@snorrsi
Copy link
Author

snorrsi commented Feb 6, 2023

@shamruk these link only work in Unity Package Manager. The links don't work in browser, at least the branch name is not changed and remains the default one of the repository.

See https://docs.unity3d.com/Manual/upm-git.html#revision

@qingzhuozhen
Copy link
Contributor

Thanks @snorrsi for sharing the instructions for tag or branch support. I will update it in our developer doc as well. Also I add the unity package for v2.6.0-no-edm and v2.6.0-no-edm-and-dep. Hope these can help.

@shamruk
Copy link

shamruk commented Feb 7, 2023

@snorrsi Oh, didn't know that, thanks!
Anyway, I'll wait asmdef (separate task) fixed before switching to the link.

@qingzhuozhen If possible also look into openupm, pipeline looks broken because the version is the same.
https://openupm.com/packages/com.amplitude.unityplugin/?subPage=pipelines
PS openupm looks like the most professional and clean way to add extensions

@lordmortis
Copy link

Is there a version of 2.8 without EDM?

@lordmortis
Copy link

Yes! See here for info!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

6 participants