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

Library type is set incorrectly when editing #147

Closed
ZigaJavornik opened this issue Dec 28, 2024 · 9 comments
Closed

Library type is set incorrectly when editing #147

ZigaJavornik opened this issue Dec 28, 2024 · 9 comments
Assignees
Labels
bug Something isn't working

Comments

@ZigaJavornik
Copy link

Hi,

I am quite new to this platform, but I am a huge fan of sharing libraries and was very excited when I saw that Twinpack exists. I have an issue, where I want to modify my own released package. When I click the "edit" button, i get the following message:

image

I then opened Twinpack, and saw the image (yes, it is silly) was updated, but notes were not added, for example. Not sure if I can provide some more information, let me know. Also adding Twinpack console output

INFO Modifying package
INFO Updating general package
INFO Updating package version
INFO Updating package configuration <path_to_project>.Zeugwerk\config.json

@iadonkey
Copy link
Contributor

iadonkey commented Dec 28, 2024

I like the logo :D

  • Some of the fields are for the package as a whole (license, project url etc.), others for a specific release of this package (version, notes and some of the fields in the „advanced“ tab). So at the moment you can’t update the notes of a release with this gui. We can change the text for you if you want though, just send it here and we‘ll update it.
  • regarding the message box: In the advanced panel. Do you have „compiled library“ checked? We don’t support this for public repos, it‘s a feature for our customers

In general, you might want to have a look at https://github.com/Zeugwerk/Twinpack-Registry , it makes releasing on Twinpack simpler by directly scanning the repository listed in https://github.com/Zeugwerk/Twinpack-Registry/blob/main/repositories.txt for new (GitHub) releases: if your repo has a GitHub release, exactly one .library file in the assets with a project version not yet available on Twinpack, a GitHub action automatically pushes the release to Twinpack.

@iadonkey
Copy link
Contributor

iadonkey commented Dec 28, 2024

Just saw that your release contains several .library files. I am not sure if Twinpack Registry is supporting this at the moment, but if not, I‘d test/implement this if you added your repo to Twinpack Registry

@ZigaJavornik
Copy link
Author

This makes sense then, just the single window with the error messages makes the user believe that modification failed. If you can, modify the "notes" to :
"You can find an example project at: https://github.com/ZigaJavornik/ZigaLibs/tree/master/Examples/Logging"

I can't seem to see a check box for compiled library in the advanced tab. There is only "type" of library, and it is set to "library" as opposed to "compiled library", which is the other option. I think this might be a bug - the field is not editable, there were no errors when I published it.
image

As far as automatic releasing goes, I keep floating between the idea of having 1 repo with multiple libraries, or split repositories with individual library releases... Should I swap to the latter, I will set up the automatic scanning and releasing from the released GitHub file.

Thanks for the prompt reply.

@iadonkey
Copy link
Contributor

iadonkey commented Dec 29, 2024

I’ll look into this bug

For our own libraries (e.g. Zeugwerk Framework libraries, which consist of ZCore, ZPlatform, ZAux, ZEquipment, ZApplication), we have a repo for every library and a “framework” repo, which has the library repos as submodules. The framework repos also contains examples, documentation (the non-api-reference one), installer scripts etc. this works quite nicely if you have CI to automate the overhead you get by doing this

@iadonkey iadonkey changed the title Issues when modifying a published package Library type is set incorrectly when editing Dec 29, 2024
@iadonkey iadonkey self-assigned this Dec 29, 2024
@iadonkey iadonkey added the bug Something isn't working label Dec 29, 2024
@ZigaJavornik
Copy link
Author

I’ll look into this bug

For our own libraries (e.g. Zeugwerk Framework libraries, which consist of ZCore, ZPlatform, ZAux, ZEquipment, ZApplication), we have a repo for every library and a “framework” repo, which has the library repos as submodules. The framework repos also contains examples, documentation (the non-api-reference one), installer scripts etc. this works quite nicely if you have CI to automate the overhead you get by doing this

I did look into submodules, and my commit history for the last day has been "add submodule->revert->add..." haha. I will take a deeper look perhaps, thanks.

@iadonkey
Copy link
Contributor

iadonkey commented Dec 29, 2024

@ZigaJavornik, could you try again please to update the note via the user interface? There was a regression on the server-side due to adding initial support for filetypes different than (.library and .compiled-library), which will be needed for Beckhoff's PLC++

Fixed with https://github.com/Zeugwerk/Twinpack-Server/commit/589912938a8fd6456f12503d77494692ba45b3fa

Would you mind if I added the Prefix "Zigas.Libs." (or similar) to your libraries, since their names (Core, Alarming, Logging) are very generic. This would only affect how the library is called in the Twinpack Catalog

Back to Twinpack Registry: Your current structure with multiple .library files per release would be supported already, so feel free to do a pull request if you want to have GitHub releases automatically on Twinpack

@ZigaJavornik
Copy link
Author

@iadonkey If I understand correctly, there is no Twinpack update for this, since it is on the server side? If that is the case, I tried and still can't edit the "notes" field.

image

As for the "prefix" of the libraries, I will add it in the project it-self, I edited the names for all 3 library names in the Twinpack GUI, it still popped up the message "compiled" not assigned, though.

@iadonkey
Copy link
Contributor

Sorry about that, deploying the server patch had an issue, now it should actually work (no client updated needed)

@ZigaJavornik
Copy link
Author

Sorry about that, deploying the server patch had an issue, now it should actually work (no client updated needed)

Indeed it does, we can close this now :)

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

2 participants