upm support #24

starikcetin · 2019-08-14T15:28:22Z

Can we convert this into a UPM package to be used with https://github.com/mob-sakai/UpmGitExtension? I can do it for you if you don't have time.

Mathijs-Bakker · 2019-08-14T16:30:14Z

Oh! That's quite cool! I never heard of it.
+1 for the idea.
And you can contribute if you like ;)

starikcetin · 2019-08-14T16:40:01Z

Yeah I recently discovered it as well, I will see what I can do.

starikcetin · 2019-08-14T23:48:50Z

Alright so here are my current results:
I followed this article: https://www.patreon.com/posts/25070968

The UPM branch is orphan. It is created by git subtree split command.

But instead of leaving the split UPM branch as is, I soft reset to the first commit and squashed everything by amending the commit. This yielded a prettier result in my opinion.

UPM branch: https://github.com/starikcetin/Extenject/tree/upm

The problem is, all the other tags are showing up at the version select list, which are not compatible:

The reason they are not compatible is that, the UpmGitExtension plugin requires the package.json to be at the repository root, but we cannot just put it in there since we need to only publish the Plugins/Zenject folder as the package.

This is not a huge deal of course, since it rejects you from importing incompatible versions (i.e. when it cannot find the package.json at the repo root). But I think we should find a better way.

Maybe we can maintain a separate repository just for the UPM releases?

Or, maybe we can implement some tag-filtering mechanism in the UpmGitExtension plugin itself, so that it only shows compatible tags. I looked at it for a while but the code seems very complicated.

starikcetin · 2019-08-15T19:30:01Z

Alright, can you give me an orphan branch named "upm" here so I can send a pull request @svermeulen ?

mob-sakai · 2019-11-26T01:10:42Z

Hi @starikcetin

I released UpmGitExtension v1.0.0 with tag filter support.
This is just a short note to inform you. ;)

starikcetin · 2019-11-26T01:55:19Z

Hi @starikcetin

I released UpmGitExtension v1.0.0 with tag filter support.
This is just a short note to inform you. ;)

you know what? you are amazing. :)

Mathijs-Bakker · 2019-12-03T22:31:05Z

@starikcetin Thanks for your contribution! It's much appreciated.
I like the fact that minor updates can be easily distributed to the users of Zenject.

UPM is an awesome thing for the future. However it's still in beta... If you've watched the Unite's Copenhagen there's a lot of new features coming to UPM. Like Asset Store Support, better Git support, etc. But the Package Developer tool is still not available.

The ultimate goal would be to have Zenject/Extenject in the UPM itself. And how cool would it be to have it in the Packages folder!

I am happy to create an UPM branch. But before that... could you enlighten me a bit more?

The package installs into the same Plugins/Zenject directory?
How does the GitHub link being advertised/registered? Does it automatically show up in the Package Manager? Is there any further action needed?
Can updates be made simpler/automated?
Does UPM automatically detects updates?
Is there anything what can cause a bad user experience?

Don't want to be a pain but just want to get these stuff clear ;)

Mathijs-Bakker · 2019-12-04T11:30:19Z

Btw the package validator gives an error:

Cannot perform upm operation: Unable to add package [https://github.com/starikcetin/Extenject/tree/upm]:
  Package name 'https://github.com/starikcetin/Extenject/tree/upm' is invalid. [InvalidParameter]

dustinlacewell · 2019-12-08T20:05:42Z

Just wanna chime in that I was able to install Extenject from https://github.com/starikcetin/Extenject.git#b2746939227c47de4b00de2a0f772df3d7831e6

starikcetin · 2019-12-08T20:56:14Z

Just wanna chime in that I was able to install Extenject from https://github.com/starikcetin/Extenject.git#b2746939227c47de4b00de2a0f772df3d7831e6

hey, you can just use https://github.com/starikcetin/Extenject.git#9.1.0, tag names are also valid.

starikcetin · 2019-12-10T03:02:31Z

@Mathijs-Bakker I'll write a detailed response when I get the chance. I've been very busy lately. But shortly, UPM is much better than any other asset sharing method I've used (and I used quite a lot of them). It keeps the assets folder clean of third party imports, provides version based updates with the click of a button, and it supports dependencies!

umfahrem · 2020-01-08T10:33:55Z

@Mathijs-Bakker
As this has been uncommented for a while let me try and address your questions:

* The package installs into the same Plugins/Zenject directory?

No, the plugin installs into a Package/Extenject directory. Since the plugin name is pulled from the package defintion this becomes the new diretory name. Also Unity handles upm packages differently and stores them outside the "Assets" directory.

* How does the GitHub link being advertised/registered? Does it automatically show up in the Package Manager? Is there any further action needed?

The package manager does by default not show existing third party projects. Users will need to add the link by themselves.

* Can updates be made simpler/automated?

Yes. I in fact did this via GitHub actions. The workflow implemented on my fork creates a new commit on an orphan upm branch and tags it with a separate upm version tag once a new version tag with the existing semantic versioning theme is pushed to the repo. This approach sadly discards any history as it creates an orphan branch and also creates the need to create separate upm tags but eliminates the need for a second repository compared to the subtree approach.

* Does UPM automatically detects updates?

If you just specify a branch it will always get the head version of that branch. If you use the upm-extension by @mob-sakai it should be able to detect the most recent version. ~~In the newest version it would not detect earlier versions for some reason though~~ (nvm, it detects them fine after adding it, you can downgrade to earlier versions). If you're not using the extension I think you have to manually delete the reference to the used version from the manifest.json.

* Is there anything what can cause a bad user experience?

As previously stated to use the ui you pretty much should use the upm-extension.

Please feel free to correct me if i missed something @starikcetin :)

Mathijs-Bakker · 2020-01-08T11:07:33Z

Hi @umfahrem
Thank you for your reply.
There was a bug with UpmGitExtension. Which didn't allow me to install a git upm.
mob-sakai/UpmGitExtension#73
I will check it later if this has been resolved.

However. UPM should be supported. And actually I have a Zenject UPM version locally. Following Unity's package development recommendations.
I ended up with some optional packages.

Zenject - DI framework
Signals (from optional extras)
Example demos (from optional extras)
Moq Extension (from optional extras)
NSubstitute Extension (from optional extras)

There are still some issues on Unity's side. Because they are still developing their UPM support. There's still something with the dependencies field in the package.js manifesto.

The last months of 2019 have been very busy for me to catch up with a lot of things. And this is one of them. I will see if I can get to it to push the local packages to the according tree.

Meanwhile just keep using the @starikcetin one.

paulpach · 2020-01-16T14:36:02Z

I tried the upm package from @starikcetin but I keep getting spammed with the following warning:

A meta data file (.meta) exists but its asset 'Packages/com.svermeulen.extenject/OptionalExtras/TestFramework/Editor/AutoSubstitute/__MACOSX/AutoSubstitute/NSubstitute' can't be found. When moving or deleting files outside of Unity, please ensure that the corresponding .meta file is moved or deleted along with it.

paulpach · 2020-01-19T19:18:06Z

Note these warnings are usually printed only once if the code is inside your project but in upm packages they keep getting printed over and over.

Mathijs-Bakker · 2020-01-19T19:20:52Z

Note these warnings are usually printed only once if the code is inside your project but in upm packages they keep getting printed over and over.

Yes you right! that's annoying.
More reason to raise this issue on @starikcetin fork

paulpach · 2020-01-19T19:21:39Z

@Mathijs-Bakker I can't, issues are disabled there, and he is not responding

Mathijs-Bakker · 2020-01-19T19:30:25Z

@Mathijs-Bakker I can't, issues are disabled there, and he is not responding

That's bad. However... did you tried to delete the .meta file?
You only see them in your systems file browser window.
And you can open the current directory from right clicking on a file in Unity's project window.
Hope this helps.

starikcetin · 2020-01-19T22:01:23Z

Hey, sorry I have been very busy with work and school lately. I will look into it when I get the chance. @paulpach

MeikelLP · 2020-02-10T08:00:43Z

@starikcetin Thank you for also pushing it to openupm :)

MeikelLP · 2020-02-10T08:03:29Z

Getting 2 warnings when installing (via openupm) and every time Unity re-focuses:

A meta data file (.meta) exists but its asset 'Packages/com.svermeulen.extenject/OptionalExtras/TestFramework/Editor/AutoSubstitute/__MACOSX/AutoSubstitute/NSubstitute' can't be found. When moving or deleting files outside of Unity, please ensure that the corresponding .meta file is moved or deleted along with it

Couldn't delete Packages/com.svermeulen.extenject/OptionalExtras/TestFramework/Editor/AutoSubstitute/__MACOSX/AutoSubstitute/NSubstitute.meta because it's in an immutable folder.

paulpach · 2020-02-13T14:28:02Z

@starikcetin any progress? that warning is so obnoxious

modesttree#24 (comment)

starikcetin · 2020-02-17T21:48:30Z

Can you try 9.1.0-1.metaFileFix version? The warning is gone on my machine.
openupm add com.svermeulen.extenject@9.1.0-1.metaFileFix
@paulpach @MeikelLP

MeikelLP · 2020-02-17T22:03:36Z

@starikcetin looks good to me

MeikelLP · 2020-02-17T22:06:42Z

Another issue I have related to UPM:

I cannot use moq.dll right now. I'm using Unity 2019.3.1f1 and set the assembly overrides manually:

But I cannot use moq or NSubstitute at all. This might not be an issue with Extenject itself but with the latest Unity package manager. I had a similar bug the other day with testables in the new inputsystem: Unity-Technologies/InputSystem#989 (comment)

Can somebody confirm that this is not possible as of right now?

MeikelLP · 2020-02-17T22:21:23Z

I can confirm that this is not related to Extenject. Unity generates wrong .csproj files (?)

My references land inside MyProject.Items.csproj instead of MyProject.Items.Tests.csproj -_-

Mathijs-Bakker · 2020-02-17T23:35:25Z

Maybe a dumb question: But does adding Zenject-usage.dll to the assembly refs help?

MeikelLP · 2020-02-18T16:27:05Z

@Mathijs-Bakker seems to be (another) internal bug in Unity. It's related to

"defineConstraints": [
    "UNITY_INCLUDE_TESTS"
],

if this is not set the dll references work fine. I opened an issue at Unity (Case 1220959). Let's see how it goes.....

MeikelLP · 2020-02-19T20:53:29Z

So apparently Moq.dll (which I want to use) is not supported in play mode (?).
Do you know if this is something this package can fix or is it an actual Unity issue?

shiena · 2020-03-09T10:03:24Z

You can use path query parameter from unity2019.3.4f1 and unity2020.1a21.
https://forum.unity.com/threads/some-feedback-on-package-manager-git-support.743345/#post-5425311

Since package.json already exists, it can be set as follows.

{
  "dependencies": {
    "com.svermeulen.extenject": "https://github.com/svermeulen/Extenject.git?path=/UnityProject/Assets/Plugins/Zenject",
    ...
  }
}

shiena · 2020-03-15T14:11:50Z

When Extenject is installed from Package Manager using path query parameter, the following sample scene cannot be opened.

If you add a samples section to package.json like ShaderGraph, it will be expanded to Assets folder, but an error will occur because the script is duplicated.
https://github.com/Unity-Technologies/Graphics/blob/v7.2.1/com.unity.shadergraph/package.json#L11-L16

If you put sample projects in a hidden folder like Sample~ in upm, it works well, but it will not be included in unitypackage.

Do you have an idea that both unitypackage and upm work?

johanhelsing · 2021-02-22T15:04:59Z

If you put sample projects in a hidden folder like Sample~ in upm, it works well, but it will not be included in unitypackage.

@shiena I made a PR #220 going right into that trap. Perhaps it could work to rename the folder removing the tilde from the .unitypackage build script?

marwie · 2021-04-03T21:17:23Z

Hello, is there any chance to release it on OpenUPM to make package installation easier? Releasing it as a package makes updating 1000x easier and more reliable.

I think Unity is also hosting a version of moq via UPM (I've seen it as a dependency in a tiny sample project recently) so adding dependencies like that could be done via UPM as well / OpenUPM

MeikelLP · 2021-04-03T23:09:05Z

Correct me if I am wrong but isn't there already OpenUPM support?
https://openupm.com/packages/com.svermeulen.extenject/

Or is that another repo?

Mathijs-Bakker · 2021-04-03T23:16:48Z

Yes, it's a fork of Extenject, forked and maintained by starikcetin

chrPetry · 2021-07-21T08:41:48Z

Is this still active?
I would really like the solution of a simple "git-link dependency" to this.

MeikelLP · 2021-07-21T10:16:34Z

@chrPetry use https://openupm.com/packages/com.svermeulen.extenject/

chrPetry · 2021-07-21T10:21:27Z

@chrPetry use https://openupm.com/packages/com.svermeulen.extenject/

As I understand I would need to add OpenUPM with this solution, right?
I would rather not use OpenUPM but the built-in Unity Package Manager.

starikcetin · 2021-07-21T13:47:00Z

@chrPetry use https://openupm.com/packages/com.svermeulen.extenject/

As I understand I would need to add OpenUPM with this solution, right?
I would rather not use OpenUPM but the built-in Unity Package Manager.

OpenUPM is external, it doesn't replace UPM. It is just a tool that automates what you would be doing manually.

My fork is a little outdated however, I don't know when I would get the chance to update it, so be aware.

Mathijs-Bakker · 2021-07-21T15:18:08Z

Unity Package Installer would be a more suitable name.

Unlike its name suggests wat it should do, Unity Package Manager (UPM) does not manage packages for you.
UPM only installs and uninstalls packages.

At Unite 2019 in Copenhagen, the UPM dev team presented the upcoming UPM feats. and their package developer tool. Then it took just a few months before the UPM project was canceled by the board. There are no exact reasons given why this happened (to the outside world). But it has to do something with the fact that UPM would become a competitor of the Unity Asset Store. And Unity needs to make money somewhere.

No other option than OpenUPM

If you want real package managing behavior, as auto update, revert back to earlier versions (tags) then OpenUPM is the only option.

To be continued

Stay tuned...

chrPetry · 2021-07-22T08:17:50Z

Alright, then thanks for this info!

shiena · 2021-07-22T10:23:22Z

upm can also be packaged in a tarball with npm pack, so it can be published to releases as well as unitypackage.
https://docs.unity3d.com/Manual/upm-ui-tarball.html

for example

MeikelLP · 2021-07-22T11:24:42Z

@Mathijs-Bakker I think the name is somewhat suited. As people can't host packages that are natively supported by UPM, OpenUPM is a free registry users can (with some work involved) upload packages which are free for everybody.

It is very sad that once again Unity is doing stuff very closed and not open to the public.

denravonska · 2022-11-04T05:51:28Z

What's the status of this? The install documentation hints at UPM being an option, but using that installs 9.2.0 whereas we're now at 9.3.1.

johanhelsing · 2022-11-04T08:14:23Z

What's the status of this? The install documentation hints at UPM being an option, but using that installs 9.2.0 whereas we're now at 9.3.1.

After modest tree won the lawsuit and got control of this repo, they basically just changed the branding and stopped maintaining the repo.

Instaling from @Mathijs-Bakker s maintained fork by git url seems like the best option https://github.com/Mathijs-Bakker/Extenject It's what I'm doing at least

denravonska · 2022-11-04T12:08:40Z

Ah, sorry. I thought I was in the fork :D
It seems to have the same problem but I'll head over there instead.

starikcetin mentioned this issue Aug 15, 2019

Tag filtering mob-sakai/UpmGitExtension#45

Closed

Mathijs-Bakker mentioned this issue Jan 19, 2020

NSubstitute warnings with upm #60

Closed

Mathijs-Bakker added the enhancement New feature or request label Jan 21, 2020

starikcetin added a commit to starikcetin/Extenject that referenced this issue Feb 17, 2020

fix for meta file warning (v: 9.1.0-1.metaFileFix)

4d90da4

modesttree#24 (comment)

starikcetin mentioned this issue Mar 9, 2020

(future plan) retire maintenance of this fork starikcetin/Extenject#5

Open

Bullrich mentioned this issue Jun 9, 2020

Unity Package Manager fix #150

Closed

23 tasks

NoTuxNoBux mentioned this issue Jan 18, 2021

Disabling Reflection Baking does not stop UWP warnings #208

Open

johanhelsing mentioned this issue Feb 22, 2021

Move sample projects to Samples~ folder so they're kept out of builds #220

Closed

23 tasks

modesttree deleted a comment from starikcetin Apr 5, 2021

upm support #24

upm support #24

Comments

starikcetin commented Aug 14, 2019

Mathijs-Bakker commented Aug 14, 2019

starikcetin commented Aug 14, 2019

starikcetin commented Aug 14, 2019 • edited Loading

starikcetin commented Aug 15, 2019 • edited Loading

mob-sakai commented Nov 26, 2019

starikcetin commented Nov 26, 2019

Mathijs-Bakker commented Dec 3, 2019

Mathijs-Bakker commented Dec 4, 2019

dustinlacewell commented Dec 8, 2019

starikcetin commented Dec 8, 2019

starikcetin commented Dec 10, 2019

umfahrem commented Jan 8, 2020 • edited Loading

Mathijs-Bakker commented Jan 8, 2020 • edited Loading

paulpach commented Jan 16, 2020

paulpach commented Jan 19, 2020

Mathijs-Bakker commented Jan 19, 2020

paulpach commented Jan 19, 2020

Mathijs-Bakker commented Jan 19, 2020

starikcetin commented Jan 19, 2020 • edited Loading

MeikelLP commented Feb 10, 2020

MeikelLP commented Feb 10, 2020 • edited Loading

paulpach commented Feb 13, 2020

starikcetin commented Feb 17, 2020 • edited Loading

MeikelLP commented Feb 17, 2020

MeikelLP commented Feb 17, 2020

MeikelLP commented Feb 17, 2020

Mathijs-Bakker commented Feb 17, 2020 • edited Loading

MeikelLP commented Feb 18, 2020

MeikelLP commented Feb 19, 2020

shiena commented Mar 9, 2020

shiena commented Mar 15, 2020 • edited Loading

johanhelsing commented Feb 22, 2021

marwie commented Apr 3, 2021 • edited Loading

MeikelLP commented Apr 3, 2021

Mathijs-Bakker commented Apr 3, 2021

chrPetry commented Jul 21, 2021

MeikelLP commented Jul 21, 2021

chrPetry commented Jul 21, 2021

starikcetin commented Jul 21, 2021 • edited Loading

Mathijs-Bakker commented Jul 21, 2021

Unity Package Installer would be a more suitable name.

No other option than OpenUPM

To be continued

chrPetry commented Jul 22, 2021

shiena commented Jul 22, 2021 • edited Loading

MeikelLP commented Jul 22, 2021

denravonska commented Nov 4, 2022

johanhelsing commented Nov 4, 2022

denravonska commented Nov 4, 2022 • edited Loading

starikcetin commented Aug 14, 2019 •

edited

Loading

starikcetin commented Aug 15, 2019 •

edited

Loading

umfahrem commented Jan 8, 2020 •

edited

Loading

Mathijs-Bakker commented Jan 8, 2020 •

edited

Loading

starikcetin commented Jan 19, 2020 •

edited

Loading

MeikelLP commented Feb 10, 2020 •

edited

Loading

starikcetin commented Feb 17, 2020 •

edited

Loading

Mathijs-Bakker commented Feb 17, 2020 •

edited

Loading

shiena commented Mar 15, 2020 •

edited

Loading

marwie commented Apr 3, 2021 •

edited

Loading

starikcetin commented Jul 21, 2021 •

edited

Loading

shiena commented Jul 22, 2021 •

edited

Loading

denravonska commented Nov 4, 2022 •

edited

Loading