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

cronie: useless (?) error message in logcat #6460

Closed
jer194 opened this issue Feb 26, 2021 · 4 comments
Closed

cronie: useless (?) error message in logcat #6460

jer194 opened this issue Feb 26, 2021 · 4 comments

Comments

@jer194
Copy link
Contributor

jer194 commented Feb 26, 2021

Problem description
When running termux crond (from cronie), logcat shows this every minute:

02-26 19:00:01.663 30231 30231 E crond   : (CRON) STAT FAILED (/data/data/com.termux/files/usr/etc/cron.d): No such file or directory
02-26 19:00:01.663 30231 30231 E crond   : (CRON) OPENDIR FAILED (/data/data/com.termux/files/usr/etc/cron.d): No such file or directory

cron itself works, using entries entered via crontab.

Steps to reproduce

  • run crond
  • check logcat

Expected behavior
Remove those logcat error logs; they seem to be useless / misleading

Additional information

@jflte:~/ termux-info
Packages CPU architecture:
arm
Subscribed repositories:
# sources.list
deb https://termux.org/packages/ stable main
# game-repo (sources.list.d/game.list)
deb https://grimler.se/game-packages-24 games stable
# science-repo (sources.list.d/science.list)
deb https://grimler.se/science-packages-24 science stable
Updatable packages:
All packages up to date
Android version:
9
Kernel build information:
Linux jflte 3.4.112 #2 SMP PREEMPT Sun Feb 7 11:42:26 CET 2021 armv7l Android
Device manufacturer:
Samsung
Device model:
GT-I9505

@ghost
Copy link

ghost commented Feb 26, 2021

Wontfix. Package customization requests are not accepted.

It informs that directory /data/data/com.termux/files/usr/etc/cron.d is not available. If this message is useless for you, then just don't read it.

@ghost ghost closed this as completed Feb 26, 2021
@ghost ghost added invalid wontfix Issue won't be fixed. labels Feb 26, 2021
@Grimler91
Copy link
Member

@jer194 have you tried manually creating the directory?

@jer194
Copy link
Contributor Author

jer194 commented Feb 26, 2021

@Grimler91:
I did that now: mkdir /data/data/com.termux/files/usr/etc/cron.d

And indeed, the error logs are gone.

But: that's in /usr/, and I don't think users should mess around there.
Furthermore, isn't it easy to fix on installation of cronie (to create that directory)?
Anyway, I agree that's minor, and I can ignore it, now that I know.
For sure there are more serious things to fix.

(Still my attitude would be different, and I'd fix it, if I could: other people might wonder as well,
and piling up such things makes it more and more difficult to spot real problems).

And actually, isnt it trivial to fix?
https://github.com/termux/termux-packages/blob/master/packages/cronie/build.sh

termux_step_create_debscripts() {
	cat <<- EOF > ./postinst
	#!$TERMUX_PREFIX/bin/sh
	mkdir -p $TERMUX_PREFIX/var/run
	mkdir -p $TERMUX_PREFIX/var/spool/cron
        mkdir -p $TERMUX_PREFIX/etc/cron.d   # add this
	EOF
}

I have submitted the patch here:
master...jer194:patch-1

(my first such thing ...)

jer194 added a commit to jer194/termux-packages that referenced this issue Feb 26, 2021
ghost pushed a commit that referenced this issue Feb 26, 2021
amuramatsu pushed a commit to amuramatsu/termux-packages that referenced this issue Jun 26, 2021
@ghost ghost locked and limited conversation to collaborators Oct 9, 2021
@xtkoba
Copy link
Contributor

xtkoba commented Oct 17, 2022

Already fixed.

@xtkoba xtkoba removed invalid wontfix Issue won't be fixed. labels Oct 17, 2022
This issue was closed.
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

No branches or pull requests

3 participants