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

Project file is modified only when there are changes #56

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

Conversation

gpambrozio
Copy link

At least for the latest XCode (11.5) XUnique seems to trigger a reload of the project file any time it runs, breaking code completion and bringing my computer to its knees every time I build as it then has to rebuild.

This seems to be because currently the file is changed in place and then reverted back to the old one if there are no changes. This intermediate change seems to get XCode to think the project changed even though XUnique reverts the change.

My changes make the script write the new project file to a separate file first and then only copies over to the project file if there are changes.

At least for the latest XCode (11.5) XUnique seems to trigger a reload of the project file any time it runs, breaking code completion and bringing my computer to its knees every time I build as it then has to rebuild.

This seems to be because currently the file is changed in place and then reverted back to the old one if there are no changes. This intermediate change seems to get XCode to think the project changed even though XUnique reverts the change.

My changes make the script write the new project file to a separate file first and then only copies over to the project file if there are changes.
@JanNash
Copy link

JanNash commented Nov 14, 2020

For anyone interested, until this PR gets merged, you can install a version of xunique including the changes from this PR and the changes from #59 via git+https://github.com/Tonband/[email protected]#egg=xunique

@JanNash
Copy link

JanNash commented Nov 14, 2020

Also, thanks @gpambrozio for fixing this, I took the liberty of pulling your commit into our repo

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.

2 participants