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 mail.py - correct reference to replacement. #1900

Open
wants to merge 1 commit into
base: main
Choose a base branch
from

Conversation

jalogisch
Copy link

make this script usable again.

this change added the ability to include this script again without touching all scripts that include this - like owncloud-unlockadmin.sh. I guess without this change those scripts would harm more than that it helps.

make this script usable again.

this change added the ability to include this script again without touching all scripts that include this - like `owncloud-unlockadmin.sh`. I guess without this change those scripts would harm more than that it helps.
@jvolkenant
Copy link
Contributor

good catch

@JoshData
Copy link
Member

JoshData commented Jan 30, 2021

It depends on where you run the script from. All of the scripts generally assume you're in the main mailinabox directory.

@@ -1,3 +1,3 @@
#!/bin/bash
# This script has moved.
management/cli.py "$@"
../management/cli.py "$@"
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
../management/cli.py "$@"
$(cd "$(dirname "${BASH_SOURCE[0]}")" && pwd)/../management/cli.py "$@"

This solves the problem of cwd. I believe this is idiomatic bash for getting the directory in which a script resides.

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