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

Opening UE4 will cause various INI files to be checked out even though they are unchanged #41

Open
gkoreman opened this issue Dec 20, 2017 · 7 comments

Comments

@gkoreman
Copy link

Simply opening the editor will cause DefaultGame.ini to be checked out even though no changes are made to the file.

@SRombauts SRombauts self-assigned this Dec 30, 2017
@SRombauts
Copy link
Owner

Hi @gkoreman, I've just tried this in UE4.18 with latest plugin version.

I do not have ini files auto-checked-out. In fact I even have a warning telling me an INI file is not checked-out.
image

But it is clear that Unreal Engine is sometimes modifying those files whenever you open the corresponding editor if it "wants" to change a configuration option (like changing it's location in the file, adding a blank line at the end...)
=> then the file is modified but not checked out "?" icon

This is not a behavior I can change in the plugin, sorry.
Regards,
Sébastien

@Ecnassianer
Copy link

I saw this issue on almost every checkin when I was collaborating with someone on a mac. Each of our copies of unreal wanted to fix up the line endings in these files, so we'd just battle back and forth switching between mac and windows EOLs. Technically the files were different, but it was annoying either way.

@SRombauts
Copy link
Owner

Once again, this is not something I can do anything about, you should post a bug report to EpicGames and let us know how it goes

@Ecnassianer
Copy link

Does this bug not happen in the perforce plugin because the file is read-only, so Unreal can never write the changed EOL version of the ini to disk?

@SRombauts
Copy link
Owner

SRombauts commented Feb 2, 2018

Okay, right, I didn't though about that one, good catch!

So in this case, the solution would be to set the Config/ files to read only, that is, lock them like Content/ in the Plastic SCM server's lock.conf file

Edit: in this case, I might need to add an option to tell the plugin to treat those files in a special manner, possibly, so I reopen this as an issue/feature request

@gkoreman
Copy link
Author

gkoreman commented Feb 2, 2018

@SRombauts I would recommend not using lock.conf for the config files, as these files can and should be merged when legitimate changes occur. I would instead recommend the readonly.conf file, briefly mentioned here: http://blog.plasticscm.com/2013/06/plastic-scm-4110445-external-release-is.html

Does anyone else have experience using readonly.conf?

@reveracorp
Copy link

I'm interested in matter as well, the DefaultGame.ini causes some confusion for members of our team.

What is the suggested best practice?

Best,

SRombautsU pushed a commit that referenced this issue Oct 13, 2022
1001844 Fix checkconnection using the parameter --server=ServerUrl
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants