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

notcurses 2.3.12 (new formula) #82112

Closed

Conversation

dankamongmen
Copy link
Contributor

  • Have you followed the guidelines for contributing?
    -Yes, including the line

    We frown on authors submitting their own work unless it is very popular.

    I asked for a clarification regarding this at how popular is "very popular"? discussions#1893, and believe it not to be an issue.

  • Have you ensured that your commits follow the commit style guide?

  • Have you checked that there aren't other open pull requests for the same formula update/change?

  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?

  • Is your test running fine brew test <formula>, where <formula> is the name of the formula you're submitting?
    -Yes, but I had to do two gross things. I export a TERM of xterm, as otherwise there is no TERM. I ought be pulling in TERM from the environment, where it ought always be set. Two, I redirect stdout (not stderr) to /dev/null, as otherwise the test process blows up (notcurses-tester generates a lot of output).

  • Does your build pass brew audit --strict <formula> (after doing brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?
    -No. I get the output Dependency 'ncurses' is provided by macOS; please replace 'depends_on' with 'uses_from_macos'. I want ncurses from homebrew, though. Is this the incorrect way to specify that? I'm using uses_from_macos for zlib, as you can see.


@BrewTestBot BrewTestBot added the new formula PR adds a new formula to Homebrew/homebrew-core label Jul 29, 2021
@carlocab carlocab changed the title initial formula for Notcurses (2.3.12) notcurses 2.3.12 (new formula) Jul 29, 2021
@dankamongmen
Copy link
Contributor Author

The CI seems to be complaining about the ncurses bit as well, but I definitely want the one from Homebrew. The macOS version is too old. The vim.rb formula seems to depend on "ncurses" just fine...

@dankamongmen dankamongmen force-pushed the dankamongmen/notcurses branch from 8f30c33 to a9dbdd8 Compare July 29, 2021 09:12
@dankamongmen dankamongmen force-pushed the dankamongmen/notcurses branch from a9dbdd8 to 819d901 Compare July 29, 2021 13:14
@dankamongmen dankamongmen force-pushed the dankamongmen/notcurses branch from 819d901 to 82a5a98 Compare July 29, 2021 13:56
@dankamongmen
Copy link
Contributor Author

still unsure what to do about this

==> Installing 'bundler' gem
notcurses:
  * Dependency 'ncurses' is provided by macOS; please replace 'depends_on' with 'uses_from_macos'.
Error: 1 problem in 1 formula detected
==> brew bottle --verbose --json notcurses --only-json-tab
==> brew bottle --merge --write --no-commit ./notcurses--2.3.12.arm64_big_sur.bottle.json
==> brew uninstall --force notcurses
==> brew uninstall --force asciidoc asciidoctor autoconf autoconf-archive automake bison boost cabal-install cargo-c cmake docbook docbook-xsl doctest doxygen ghc gnu-getopt gnu-sed libatomic_ops libgit2 libssh2 llvm meson nasm ninja pandoc rust source-highlight sphinx-doc swig util-macros xcb-proto xmlto xtrans yasm
==> brew install --only-dependencies ./notcurses--2.3.12.arm64_big_sur.bottle.tar.gz
==> brew install ./notcurses--2.3.12.arm64_big_sur.bottle.tar.gz
==> brew linkage --test notcurses
==> brew install --only-dependencies --include-test notcurses
==> brew test --verbose notcurses
==> FAILED
==> Testing notcurses
/usr/bin/sandbox-exec -f /private/tmp/homebrew20210729-77855-1u4t9hr.sb ruby -W1 -- /opt/homebrew/Library/Homebrew/test.rb /opt/homebrew/Library/Taps/homebrew/homebrew-core/Formula/notcurses.rb --verbose
==> /opt/homebrew/Cellar/notcurses/2.3.12/bin/notcurses-info
Error: test failed
dyld: Library not loaded: @rpath/libnotcurses.2.dylib
  Referenced from: /opt/homebrew/Cellar/notcurses/2.3.12/bin/notcurses-info
  Reason: image not found
Error: notcurses: failed

is the "image not found" related to the early failure? and how do i work around this to require the Brew of ncurses? i seem to be doing the same thing as the vim package.

@carlocab carlocab added the CI-force-linux [DEPRECATED] Don't pass --skip-unbottled-linux to brew test-bot. label Jul 29, 2021
@SMillerDev
Copy link
Member

It fails to load it's own library from the rpath. It's unrelated to the ncurses audit above.

@dankamongmen dankamongmen force-pushed the dankamongmen/notcurses branch from 8ae0004 to 495001a Compare July 29, 2021 14:35
carlocab
carlocab previously approved these changes Jul 29, 2021
Copy link
Member

@carlocab carlocab left a comment

Choose a reason for hiding this comment

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

LGTM, provided the only failure remaining is the audit failure.

Thanks for your contribution to Homebrew, @dankamongmen.

@carlocab
Copy link
Member

Oops, Linux will fail as we haven't bottled ffmpeg yet. We can ignore that.

@dankamongmen
Copy link
Contributor Author

LGTM, provided the only failure remaining is the audit failure.

Thanks for your contribution to Homebrew, @dankamongmen.

no, thank you for the opportunity and feedback, and for making open source available to macOS users!

i will diligently maintain this Formula as i do all my other packages elsewhere.

@carlocab
Copy link
Member

Great. You probably won't need to do much as long as you don't change your build system or your dependencies. For version bumps, you might be interested in brew bump-formula-pr, or https://github.com/dawidd6/action-homebrew-bump-formula.

@carlocab carlocab removed the CI-force-linux [DEPRECATED] Don't pass --skip-unbottled-linux to brew test-bot. label Jul 29, 2021
Signed-off-by: Nick Black <[email protected]>
Co-authored-by: Sean Molenaar <[email protected]>
Co-authored-by: Carlo Cabrera <[email protected]>
@dankamongmen dankamongmen force-pushed the dankamongmen/notcurses branch from 70e5e20 to a285bdb Compare July 29, 2021 15:06
@BrewTestBot
Copy link
Member

:shipit: @carlocab has triggered a merge.

@dankamongmen dankamongmen deleted the dankamongmen/notcurses branch August 5, 2021 03:56
@github-actions github-actions bot added the outdated PR was locked due to age label Sep 5, 2021
@github-actions github-actions bot locked as resolved and limited conversation to collaborators Sep 5, 2021
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. new formula PR adds a new formula to Homebrew/homebrew-core outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants