Skip to content

davmail 6.4.0 #235690

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.

Sign up for GitHub

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

Jump to bottom
Merged
merged 2 commits into from
Sep 5, 2025
Merged

davmail 6.4.0 #235690

merged 2 commits into from
Sep 5, 2025
+6 −5

Conversation

botantony
Copy link
Contributor

  • Have you followed the guidelines for contributing?
  • 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 HOMEBREW_NO_INSTALL_FROM_API=1 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?
  • Does your build pass brew audit --strict <formula> (after doing HOMEBREW_NO_INSTALL_FROM_API=1 brew install --build-from-source <formula>)? If this is a new formula, does it pass brew audit --new <formula>?

@github-actions github-actions bot added the java Java use is a significant feature of the PR or issue label Aug 31, 2025
@github-actions GitHub Actions
Copy link
Contributor

🤖 An automated task has requested bottles to be published to this PR.

Caution

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@github-actions GitHub Actions
Copy link
Contributor

⚠️ Bottle publish failed.

@daeho-ro
Copy link
Member 

environment_variables PATH: std_service_path_env

This line is a problem, and I don't think davmail will call other formula.

daeho-ro
daeho-ro approved these changes Sep 1, 2025
View reviewed changes
Copy link
Member

@daeho-ro daeho-ro left a comment

Choose a reason for hiding this comment

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

Let's try

@carlocab
Copy link
Member

carlocab commented Sep 1, 2025 


environment_variables PATH: std_service_path_env

This line is a problem, and I don't think davmail will call other formula.

If this is not being relocated then that seems like a brew bug and should be reported at Homebrew/brew.

@daeho-ro
Copy link
Member

daeho-ro commented Sep 1, 2025

It contains /usr/local/bin and that is too generic.

@carlocab
Copy link
Member

carlocab commented Sep 1, 2025

It contains /usr/local/bin and that is too generic.

These service files are generated by brew so we know they should always be relocated, even if /usr/local/bin is generic.

@daeho-ro daeho-ro marked this pull request as draft September 1, 2025 11:14
@daeho-ro daeho-ro added the brew Issue may be Homebrew/brew related label Sep 2, 2025
@daeho-ro daeho-ro mentioned this pull request Sep 3, 2025
Open
3 tasks
@botantony @daeho-ro
davmail 6.4.0
945e123 
Signed-off-by: botantony <antonsm21@gmail.com>
@daeho-ro daeho-ro marked this pull request as ready for review September 5, 2025 06:50
@github-actions github-actions bot added the automerge-skip `brew pr-automerge` will skip this pull request label Sep 5, 2025
@carlocab carlocab removed the automerge-skip `brew pr-automerge` will skip this pull request label Sep 5, 2025
@github-actions GitHub Actions
Copy link
Contributor

github-actions bot commented Sep 5, 2025

🤖 An automated task has requested bottles to be published to this PR.

Caution

Please do not push to this PR branch before the bottle commits have been pushed, as this results in a state that is difficult to recover from. If you need to resolve a merge conflict, please use a merge commit. Do not force-push to this PR branch.

@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 Sep 5, 2025
@BrewTestBot BrewTestBot added this pull request to the merge queue Sep 5, 2025
Merged via the queue into Homebrew:main with commit 2e67959 Sep 5, 2025
19 checks passed
@botantony botantony deleted the davmail branch September 5, 2025 08:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@chenrui333 chenrui333 chenrui333 approved these changes

@carlocab carlocab carlocab approved these changes

@daeho-ro daeho-ro daeho-ro approved these changes

@github-actions github-actions[bot] github-actions[bot] approved these changes

Assignees
No one assigned
Labels
brew Issue may be Homebrew/brew related build-all-bottle CI-published-bottle-commits The commits for the built bottles have been pushed to the PR branch. java Java use is a significant feature of the PR or issue
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
5 participants
@botantony @daeho-ro @carlocab @chenrui333 @BrewTestBot