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

Mac OSX + Makefile Problem with find -daytime #120

Closed
bryanhanson opened this issue May 12, 2020 · 5 comments
Closed

Mac OSX + Makefile Problem with find -daytime #120

bryanhanson opened this issue May 12, 2020 · 5 comments

Comments

@bryanhanson
Copy link
Collaborator

Looks like Apple does not implement find -daytime in any shell they offer. There are several possible workarounds that do roughly the same thing. Here's the offending line in Makefile:

DESCRIPTION: $(shell find hyperSpec -maxdepth 1 -ctime 1 -not -ctime 0 -name "DESCRIPTION") #only if not modified today

One option is to replace -daytime with -ctime 1 which checks if the file has been modified in the last 24 hrs.

bryanhanson added a commit that referenced this issue May 12, 2020
bryanhanson added a commit that referenced this issue May 12, 2020
ximeg added a commit that referenced this issue May 13, 2020
@ximeg
Copy link
Collaborator

ximeg commented May 13, 2020

Closed after merged PR #122

@ximeg ximeg closed this as completed May 13, 2020
@bryanhanson bryanhanson reopened this May 19, 2020
@bryanhanson
Copy link
Collaborator Author

We are not getting the right dates from the changes I made.

@ximeg
Copy link
Collaborator

ximeg commented May 19, 2020

Well, actually if we address the sematic versioning issue #123 , we won't need to place current date into DESCRIPTION anymore. (It is also annoying to see the hyperSpec/DESCRIPTION file changed every time I build the package - git starts to complain about this file when I want to change the branch)

@bryanhanson
Copy link
Collaborator Author

Yes, I would like to see #123 fixed sooner rather than later.

@bryanhanson
Copy link
Collaborator Author

Alright, I'll close this as it's really going to be addressed with #123.

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

No branches or pull requests

2 participants