Skip to content

Resort and extend startup log information #1556

Merged
merged 5 commits into from Jul 24, 2013

5 participants

@Karlson2k
Team Kodi member
  • Resort XBMC information (from "XBMC ... . Platform... build..." to "XBMC ... build... Platform... " )
  • Add compiler information to log
  • Add debug/release information to log
@MartijnKaijser
Team Kodi member

this seems like a good idea.give a better idea where the build comes from. opinions?

@Memphiz
Team Kodi member
Memphiz commented Jul 23, 2013

Fine by me if the VERSION define is there for all gcc platforms for sure...

@MartijnKaijser
Team Kodi member

@Karlson2k can you rebase.
@Memphiz how can we find out? Do a build run and just test them?

@Memphiz
Team Kodi member
Memphiz commented Jul 23, 2013

If it Builds it is fine

@Karlson2k
Team Kodi member

@MartijnKaijser I'll update it a little and rebase.

@Karlson2k
Team Kodi member

Now not using __VERSION__ macro

@Memphiz
Team Kodi member
Memphiz commented Jul 24, 2013

Jenkins Build this please

@Memphiz
Team Kodi member
Memphiz commented Jul 24, 2013

Pr is to old and jenkins didn't scan it - manual testbuild triggered...

@Memphiz Memphiz merged commit 0512600 into xbmc:master Jul 24, 2013
@MartijnKaijser
Team Kodi member

August starts early this year...

@Memphiz
Team Kodi member
Memphiz commented Jul 24, 2013

It's no feature - it's for helping us supporting people - no reason to suspend this until the merge window.

@Fneufneu
Team Kodi member

and if i build xbmc with clang ? :)

Team Kodi member

@Fneufneu Is it possible on some platform?

Team Kodi member

clang can be used on Linux / OSX / FreeBSD.
it will be the default compiler on (the futur) FreeBSD 10 it's why i make all my test with clang.

Team Kodi member

@Fneufneu OK, I'll extend compiler detection.

Team Kodi member

@Memphiz Does it make sense to see if we also add test compiling with clang on Jenkins (if that's possible)?

Team Kodi member

I would say no - the platforms jenkins compiles for now are compiled the way the readme suggests. I guess usage of clang would be introduced automagically when ubuntu for example switches over from gcc. In that case we would need to add buildslaves for each platform using the specific os/buildtools.

On OSX depending on the used Xcode version clang is already used IIRC. We don't add a buildslave for all Xcode versions though (as it would be a pita to compile for them). We just ensure it works whenever a new Xcode comes out and normally once it compiled this doesn't change during XBMC development.

So if fneufneu wants to have a freebsd slave and freebsd uses clang - this would be different. We could add a freebsd slave of course - that is up to fneufneu. (someone would need to setup a vm for this of course...)

Team Kodi member

clang define GCC version macros too, but 'GCC' identification in log will be wrong.
I'll correct it.

Team Kodi member

@Fneufneu Clang support #3012
Tested on Ubuntu

@MartijnKaijser
Team Kodi member

i'm not seeing the "Using foo XBMC build" in the log when using a nightly build.
@Memphiz what are our default value is when doing a build?

@Karlson2k
Team Kodi member

@MartijnKaijser It's on LOGINFO level. Only visible on debug logging or debug build.

@Karlson2k Karlson2k deleted the Karlson2k:Startup_log_01 branch Jul 26, 2013
@MartijnKaijser
Team Kodi member

And thats what I run and it's not there. It doesn't show anything and since we either do a debug release build it should be there.

@garbear
Team Kodi member
garbear commented Jul 27, 2013

looking at 3fae98a, it seems that NDEBUG isn't defined on windows release build

@Karlson2k
Team Kodi member

@MartijnKaijser @garbear Loglevel setting is initialized later, so on startup only default level is logged.
It's corrected in PR #3012

@LongChair LongChair added a commit to plexinc/plex-home-theater-public that referenced this pull request May 25, 2015
@LongChair LongChair Clear any pending fetch job upon MediaWindow update, should fix #1556 7f7a4cb
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Something went wrong with that request. Please try again.