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

Make phobos.lib include std.experimental.logger #3023

Merged
merged 1 commit into from
Feb 24, 2015
Merged

Conversation

e10s
Copy link
Contributor

@e10s e10s commented Feb 24, 2015

Note this is NOT tested on Win64 because I have no such environment.

@burner
Copy link
Member

burner commented Feb 24, 2015

what was the problem? from the diff it looked like you just moved the lines!

update:
looking at different autotester output from phobos testing, I got it.
Thanks

burner added a commit that referenced this pull request Feb 24, 2015
Make phobos.lib include std.experimental.logger
@burner burner merged commit 96e2bfe into dlang:master Feb 24, 2015
@burner
Copy link
Member

burner commented Feb 24, 2015

did not work for win64

fix: #3024

@burner
Copy link
Member

burner commented Feb 24, 2015

@MartinNowak somehow the logger was not included in the win32 and win64 build.
this is strange as win32/64 failed me regularly before I rewrote FileLogger.
Anyway, this and #3024 should fix that. IMO this needs to go into 2.067.
Sorry for the inconvenience.

@burner burner added this to the 2.067 milestone Feb 25, 2015
@MartinNowak
Copy link
Member

looking at different autotester output from phobos testing, I got it.

So what was fixed here?

@burner
Copy link
Member

burner commented Feb 26, 2015

all logger source where not include in phobos.lib and the unittest

this fixes it for win32

burner added a commit that referenced this pull request Feb 27, 2015
Make phobos.lib include std.experimental.logger
@e10s e10s deleted the liblogger branch February 20, 2018 19:46
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.

3 participants