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 to 3.08 #20

Merged
merged 12 commits into from
Nov 13, 2022
Merged

Update to 3.08 #20

merged 12 commits into from
Nov 13, 2022

Conversation

jaimergp
Copy link
Member

@jaimergp jaimergp commented Jul 7, 2022

Checklist

  • Used a personal fork of the feedstock to propose changes
  • Bumped the build number (if the version is unchanged)
  • Reset the build number to 0 (if the version changed)
  • Re-rendered with the latest conda-smithy (Use the phrase @conda-forge-admin, please rerender in a comment in this PR for automated rerendering)
  • Ensured the license file is being packaged.

Updates to latest version, bringing some more plugins so it's in sync with defaults. Future work on constructor will require this version for EXE signing.

@conda-forge-linter
Copy link

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

@jaimergp
Copy link
Member Author

jaimergp commented Jul 7, 2022

@conda-forge-admin, please rerender

@jaimergp
Copy link
Member Author

jaimergp commented Jul 7, 2022

makensis -VERSION does not error on Linux, but doesn't print the version either.

Running it locally with the downloaded artifact does print something:

$ makensis -VERSION
v07-Jul-2022.cvs(nsis) 

It's using the build date I guess?

@jaimergp
Copy link
Member Author

jaimergp commented Jul 7, 2022

The issue was already present in 3.06.1, btw:

$ makensis -VERSION
v22-Sep-2020.cvs(nsis)

@jaimergp
Copy link
Member Author

jaimergp commented Jul 7, 2022

Explicitly passing VERSION=${PKG_VERSION} works!

$ makensis -VERSION
v3.08

@jaimergp
Copy link
Member Author

jaimergp commented Jul 7, 2022

Ready for review @conda-forge/nsis!

@hmaarrfk
Copy link

Is this still blocking on: conda/constructor#526

@jaimergp
Copy link
Member Author

I issued repodata patches for constructor, but they were not merged yet. See conda-forge/conda-forge-repodata-patches-feedstock#300

@jaimergp jaimergp closed this Nov 3, 2022
@jaimergp jaimergp reopened this Nov 3, 2022
@jaimergp
Copy link
Member Author

jaimergp commented Nov 3, 2022

@conda-forge-admin, please rerender

@jaimergp
Copy link
Member Author

jaimergp commented Nov 3, 2022

@jakirkham @mbargull - this is ready for review whenever you have time :D

recipe/meta.yaml Outdated Show resolved Hide resolved
@jaimergp
Copy link
Member Author

jaimergp commented Nov 8, 2022

Gentle ping @conda-forge/nsis :D

@jaimergp
Copy link
Member Author

@conda-forge-admin, please rerender

@jaimergp
Copy link
Member Author

Looks like everything is in order. I'll merge to unblock the constructor release. The previous builds of constructor are pinned to not use 3.08 so this shouldn't break anything in the unlikely event something is off.

@jaimergp jaimergp merged commit 0fc04f4 into conda-forge:main Nov 13, 2022
folder: plugins/UAC
- url: https://github.com/mingwandroid/nsis-untgz/archive/5c814c5f2c8a9a14e3a6ddd3e594fcc76db5b86a.zip
sha256: ae68c41493abbb8800640acdf67a06c63bcceaaf21b539c50c348a20dc4b2803
folder: plugins/untgz
- url: https://github.com/mingwandroid/nsis-UnicodePathTest/archive/fa74caef553883f1820049d89e169aff57551796.zip
sha256: 1a4dc09f0fbb7d6be88835c50a7c95dbd37470c65e29cf676b14e3ad9c4d7494
folder: plugins/UnicodePathTest
# AccessControl does not provide historical versions as far as I can tell..
# this is for version v1.0.8.3 - 24th February 2021
- url: https://nsis.sourceforge.io/mediawiki/images/4/4a/AccessControl.zip
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Please upload this archive somewhere and use that instead of using an unstable link like this

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Good point. I blindly copied from the Anaconda feedstock (https://github.com/AnacondaRecipes/aggregate/blame/master/nsis-feedstock/recipe/meta.yaml#L28) but it's true we should probably mirror these like it's being done in the other cases.

- url: https://code.kliu.org/misc/elevate/elevate-1.3.0-redist.7z
sha256: b1b3f070353a0eadee2cea3a575049d10df9763ff24e39313da4cec9455382e1
folder: plugins/elevate
- url: http://nsis.sourceforge.net/mediawiki/images/7/79/UAC_v0.2.2d.zip
sha256: 9e64d93185e468fb873925db887f637778d926b864b5ff85600b7c9fce92660d
- url: https://nsis.sourceforge.io/mediawiki/images/8/8f/UAC.zip
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

same here

@jakirkham jakirkham mentioned this pull request Jan 27, 2023
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.

4 participants