Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Already on GitHub? Sign in to your account

Rearrange visual studio project #908

Merged
merged 1 commit into from May 10, 2012

Conversation

Projects
None yet
6 participants
Member

jmarshallnz commented Apr 30, 2012

Just something that's bugged me for a bit, developing on different platforms.

Up to the win32 mainstays as to whether it's pulled in.

@wsoltys, @chadoe, @CrystalP what say you?

Owner

Montellese commented Apr 30, 2012

You can't imagine how often I went into the "video" folder just to remember that VideoDatabase.cpp/.h are in "database" in the VS project but I was always too lazy to change it. I for one prefer the VS project structure to reflect the filesystem structure because it makes looking for files on linux etc easier as well.

Member

wsoltys commented Apr 30, 2012

fine by me

@ghost ghost assigned jmarshallnz Apr 30, 2012

Member

jmarshallnz commented Apr 30, 2012

Cheers - will pull on May 10 to save others (a little bit of) hassle

Contributor

DDDamian commented May 1, 2012

+1 to keeping to filesystem layout

Contributor

CrystalP commented May 1, 2012

Fine for me as well.

Member

chadoe commented May 1, 2012

Absolutely, thought I did most of it during reshuffle to match the filesystem, guess not.

Member

wsoltys commented May 1, 2012

the order got broken due to all the git merges or someone working with a text editor. current master is a mess in terms of the filters file. sources are referenced in the filter file but not shown and some filters being empty. last example is the thread refactor with commons being empty and no threads filter at all.

Member

wsoltys commented May 1, 2012

mmh last example may be wrong as it seems that common and threads are now own projects. didn't checked the merge cascade.

Member

jmarshallnz commented May 1, 2012

There was an issue yesterday (due to an = sign outside a tag) but pretty sure it's sorted now in master.

Contributor

DDDamian commented May 6, 2012

Will scramble again pretty good with AE ;)

jmarshallnz added a commit that referenced this pull request May 10, 2012

@jmarshallnz jmarshallnz merged commit 4bc43bb into xbmc:master May 10, 2012

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