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

Update FortiClient version to 7.0.1 #306

Closed
wants to merge 1 commit into from
Closed

Update FortiClient version to 7.0.1 #306

wants to merge 1 commit into from

Conversation

BJD1997
Copy link

@BJD1997 BJD1997 commented Aug 22, 2022

Tested the package locally and succesfully installs FortiClient 7.0.1
Hope the package can be updated this way to a somewhat more recent version :)

@bcurran3
Copy link
Owner

Hey BJ! Looks like you "cracked the code." Congrats.

I'm happy to merge this in and push out an update as a bunch of people want it, but before I do... wouldn't you like to be a maintainer? Then you can do it. :-)

My FortiGate F80E is has been collecting dust for a year or two and I don't have any active clients using FGs at the moment. Let me pass this on to you.

@BJD1997
Copy link
Author

BJD1997 commented Aug 23, 2022

I would like to but i'm not sure I can crack the code (yet) on the online installer (versions past 7.0.1)
How do I become the new maintainer for this package?

@bcurran3
Copy link
Owner

Just let me know your Chocolatey ID

@BJD1997
Copy link
Author

BJD1997 commented Aug 23, 2022

https://community.chocolatey.org/profiles/BJD1997
Created a Profile on the community. Is this what you need? If not where do I get my Chocolatey ID?

@bcurran3
Copy link
Owner

BJD1997 - that's it. You should get an e-mail now inviting you to become a maintainer.

@BJD1997
Copy link
Author

BJD1997 commented Aug 23, 2022

Yes I see myself as maintainer now on Chocolatey. And have replicated the package on my own respository.
https://github.com/BJD1997/ChocolateyPackages.

Can I publish the fix to my repo directly or should I change the resource url first?
Also do you still want yourself to be listed as maintainer? If not I can remove your user from the package if you want.

@bcurran3
Copy link
Owner

You should update the package metadata appropriately
You should push the update.
You should leave the maintainer list alone.

I'm going to go ahead and close this as you've now got the ability to update the package now.

@bcurran3 bcurran3 closed this Aug 23, 2022
jpluimers pushed a commit to jpluimers/bcurran3.ChocolateyPackages that referenced this pull request Oct 6, 2022
Updates .nuspect (version) and install script (download URL and sha256)
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

Successfully merging this pull request may close these issues.

2 participants