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

at: make at jobs persist #22060

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

at: make at jobs persist #22060

wants to merge 1 commit into from

Conversation

Lothia
Copy link

@Lothia Lothia commented Sep 6, 2023

This commit should make the at jobs persist across reboots just ike crontab.

I don't know why but the with the START=50 the edited init script doesn't work, setting it 99 makes it work.

Maintainer: @philenotfound
Compile tested: none
Run tested: Xiaomi Mir3g, just edited the init file, added jobs and reboot to test it.
Description:

#11321

this commit should make the at jobs persist accros reboots just ike crontab.

I don't know why but the with the START=50 the edited init script doesn't work, setting it 99 makes it work.

Signed-off-by: Lothia <35460542+Lothia@users.noreply.github.com>
@Lothia Lothia changed the title make at jobs persist at: make at jobs persist Sep 6, 2023
Copy link
Member

@jefferyto jefferyto left a comment

Choose a reason for hiding this comment

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

Also:

  • The commit message should be prefixed with the package name
  • A real name is required for the Signed-off-by line

See CONTRIBUTING.md.

mkdir -m 0755 -p /etc/cron/atspool
ln -s /etc/cron/atspool /var/spool/cron/ 2>/dev/null
chown -R nobody:nogroup /var/spool/cron/atspool
}
Copy link
Member

Choose a reason for hiding this comment

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

Please use tabs - it reduces the number of lines changed and makes the file smaller.

Copy link
Author

Choose a reason for hiding this comment

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

Thanks, looks like I have missed the tabs. Though I am pretty sure I've put tabs instead of spaces? Weird.

I am not comfortable sharing my real name.

@BKPepe
Copy link
Member

BKPepe commented Oct 7, 2023

@Lothia kind reminder otherwise it will be closed.

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.

None yet

3 participants