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

update Visual studio gitignore for qt add-in #1586

Closed
wants to merge 1 commit into from
Closed

update Visual studio gitignore for qt add-in #1586

wants to merge 1 commit into from

Conversation

AlexiaChen
Copy link

this VisualStudio.gitignore have no ignore files for qt-vs-addin. such as moc__.cpp qrc__.cpp ui_*.h

@shiftkey
Copy link
Member

shiftkey commented Jul 8, 2015

There's a couple of details I need before closing this out:

  • a link to the QT add-in for VS as an intro
  • a link to an explanation for what these entries represent, and why they should be ignored in source control

@AlexiaChen
Copy link
Author

1.Here is the wiki about QT add-in for VS :
https://wiki.qt.io/Visual_Studio_Add-in
1.
moc_*.cpp---------http://www.qtcentre.org/threads/22644-what-is-a-moc-file
moc files are just generated files by Meta-Object-Compilor of Qt. they are similar to *.obj files for temporary during building time.

qrc_*.cpp-------------here is a link to the resource system of qt: http://doc.qt.io/qt-5/resources.html
qrc files are just generated files by Qt-Resource-Compilor for temporary during building time.

ui__.h------------------here is a link to the User Interface Compiler (uic) http://doc.qt.io/qt-4.8/uic.html
ui__.h files are just generated files by UIC.exe for temporary during building time. it represent the layout of User Interface in C++

GeneratedFiles/------------This is a folder generated by Qt VS add-in. moc,qrc,ui files are all under this folder

@shiftkey
Copy link
Member

shiftkey commented Jul 8, 2015

GeneratedFiles/
This is a folder generated by Qt VS add-in. moc,qrc,ui files are all under this folder

So does this mean GeneratedFiles/ will include all the other rules?

@AlexiaChen
Copy link
Author

Reply:"So does this mean GeneratedFiles/ will include all the other rules?"

Yes,it will include all the other files I mentioned above.

@shiftkey
Copy link
Member

shiftkey commented Jul 8, 2015

Okay cool, so can we drop the other rules here and just leave GeneratedFiles/ as the only addition?

I'm not sure how I feel about linking to an SO answer which explains this rather than some official documentation, but this is a good summary:

http://stackoverflow.com/a/18930469/1363815

@arcresu thoughts?

@AlexiaChen
Copy link
Author

Yes,SO answer which explains this rather than some official documentation. I agree.

OKay,We can drop the other rules here and just leave GeneratedFiles/ as the only addition,I think it is a good idea. 👍

@arcresu
Copy link
Contributor

arcresu commented Jul 8, 2015

Official documentation is better because it somewhat constitutes a contract, but it's not realistic to expect any official docs about this stuff to exist for every tool. SO answers or even examples of file contents are a lot better than nothing when there's no obvious official source.

@shiftkey
Copy link
Member

OKay,We can drop the other rules here and just leave GeneratedFiles/ as the only addition

I'd be happy with that.

@shiftkey
Copy link
Member

shiftkey commented Mar 6, 2016

I'm trying to clear out the backlog of PRs here, and given the comments haven't seen a reply I'm going to close this out.

@AlexiaChen please let me know if you're interested in working together so this can get merged in.

@shiftkey shiftkey closed this Mar 6, 2016
@AlexiaChen
Copy link
Author

@shiftkey Yeah, I am here, I was interested in working together as always.

@AlexiaChen
Copy link
Author

@shiftkey Maybe, I need to repost a new PR for

OKay,We can drop the other rules here and just leave GeneratedFiles/ as the only addition

?

@AlexiaChen
Copy link
Author

this PR need to delete some extra rules, we just leave only one rule of "GeneratedFiles/"

@shiftkey
Copy link
Member

shiftkey commented Mar 7, 2016

@AlexiaChen that sounds good!

@AlexiaChen
Copy link
Author

@shiftkey soooo? so I need to repost a new PR for this as I mentioned above?

@shiftkey
Copy link
Member

shiftkey commented Mar 7, 2016

@AlexiaChen yes please!

@AlexiaChen
Copy link
Author

@shiftkey OK. Thank you from my heart

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants