Improvement Request about laurine HOT 5 CLOSED

Hey Edinson,
thank you for your feedback!

I have to say I already encountered exactly same thing. The issue actually runs deeper - if you are not working alone, it becomes major hinderance. There is also second problem with script that I resolved over the weekend - I thought it might be good idea to lock that file, unfortunately git then just refuses to cooperate with you completely when you need, for example, reset that file! and this goes back to each-time-regenerating file, because sometimes it goes to situation where you just can't do anything with it - reset, stash, reverse.. oh well.

So yes, I completely agree this should be changed. I think the timestamp has some meaning there, I will change the bash script so it checks and generates only if needed and add it to readme / example. You can modify following to fix it (in build phase):

if [ "$OUTPUT_PATH" -ot "$SOURCE_PATH" ]; then
    "$LAURINE_PATH" -i "$SOURCE_PATH" -o "$OUTPUT_PATH"
fi

Have a nice day!

from laurine.

Cool adding that snippet seems to do the trick, it's much better now.

Thanks!

from laurine.

Glad I could help!

from laurine.

Hi guys,

Firstly, I would like to make you aware of the options in Xcode for specifying the inputs and outputs that it uses for dependency testing. Screenshot below. These reduce unnecessary invocations of the build step.

Secondly, I very strongly disagree that any sort of timestamp should exist in the source file. This is not the first time that I have seen redundant timestamps cause churn in the VCS.

The timestamp is metadata and I believe that such meta belongs only in the VCS, not in the file content.

from laurine.

Hey Richard

I know about the first option ; I considered it and realized, that while we might know, not everyone will - and there is nothing preventing you to modify that script on your own - that is why it is just example, right.

My focus is to create something that is so easy to use, that you don't have to think about it - sure, might seem like small change, but ctrl+c with everything still seem more viable.

As for the second question, after considering all pros and cons, I have to agree with you - metadata such as this should be governed only by VCS, if the date would be date of creation (as XCode does it) that would probably be okay - but that is also pointless to do.

I will be changing the generator so it omits the date altogether and I will keep the script as we decided earlier due to simple fact that it is easy to copy paste. I might add more options in the future, I am considering ways how to make it easy using cocoapods anyway.

Thank you so much for your support.

from laurine.