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

asymptote 2.95 #198910

Merged
merged 2 commits into from
Nov 28, 2024
Merged

asymptote 2.95 #198910

merged 2 commits into from
Nov 28, 2024

Conversation

BrewTestBot
Copy link
Member

@BrewTestBot BrewTestBot commented Nov 25, 2024

Created by brew bump


Created with brew bump-formula-pr.

  • resource blocks have been checked for updates.

@github-actions github-actions bot added the bump-formula-pr PR was created using `brew bump-formula-pr` label Nov 25, 2024
@daeho-ro daeho-ro force-pushed the bump-asymptote-2.95 branch from ce885b3 to 95e0280 Compare November 25, 2024 13:37
@daeho-ro daeho-ro added the CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. label Nov 25, 2024
@daeho-ro
Copy link
Member

daeho-ro commented Nov 25, 2024

This will install symlink in the bin folder, should I remove it from the post-install step?

.rwxr-xr-x daeho.ro staff 5.8 MB Mon Nov 25 18:23:48 2024 asy
lrwxr-xr-x daeho.ro staff  38 B  Mon Nov 25 18:23:48 2024 xasy ⇒ /usr/local/share/asymptote/GUI/xasy.py

Or, maybe local patch to modify Makefile.in?

install-asy: asy sty $(PYFILES)
        ...
	${INSTALL} -p -m 644 GUI/res/icons/*.svg $(GUIdir)/res/icons
	ln -sf @datadir@/asymptote/GUI/xasy.py $(bindir)/$(XNAME)

@daeho-ro daeho-ro mentioned this pull request Nov 25, 2024
1 task
@daeho-ro daeho-ro force-pushed the bump-asymptote-2.95 branch 5 times, most recently from cc24a65 to 35cf62e Compare November 26, 2024 10:22
Formula/a/asymptote.rb Outdated Show resolved Hide resolved
Formula/a/asymptote.rb Outdated Show resolved Hide resolved
@daeho-ro daeho-ro removed the ready to merge PR can be merged once CI is green label Nov 27, 2024
@daeho-ro daeho-ro force-pushed the bump-asymptote-2.95 branch from 60572d0 to d45be15 Compare November 27, 2024 06:27
@daeho-ro daeho-ro force-pushed the bump-asymptote-2.95 branch from d45be15 to 7136c5a Compare November 27, 2024 07:32
@daeho-ro
Copy link
Member

Tried to add a permission but the file is not exist yet and don't understand how to deal with this case.
Even if I add it to the post_install, CI is also failed.

Formula/a/asymptote.rb Outdated Show resolved Hide resolved
@github-actions github-actions bot added the autosquash Automatically squash pull request commits according to Homebrew style. label Nov 28, 2024
Co-authored-by: Carlo Cabrera <[email protected]>
@daeho-ro daeho-ro force-pushed the bump-asymptote-2.95 branch from 1178425 to c363076 Compare November 28, 2024 04:57
@github-actions github-actions bot removed the autosquash Automatically squash pull request commits according to Homebrew style. label Nov 28, 2024
@daeho-ro daeho-ro added the ready to merge PR can be merged once CI is green label Nov 28, 2024
Copy link
Contributor

@github-actions github-actions bot added the CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. label Nov 28, 2024
@BrewTestBot BrewTestBot added this pull request to the merge queue Nov 28, 2024
Merged via the queue into master with commit 41977f8 Nov 28, 2024
15 checks passed
@BrewTestBot BrewTestBot deleted the bump-asymptote-2.95 branch November 28, 2024 12:33
Copy link
Contributor

⚠️ @daeho-ro pre-merge checks failed.

@daeho-ro
Copy link
Member

Run pr-publish command in my cli history mistakenly. :(

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bump-formula-pr PR was created using `brew bump-formula-pr` CI-no-fail-fast Continue CI tests despite failing GitHub Actions matrix builds. CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. ready to merge PR can be merged once CI is green
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants