Movatterモバイル変換


[0]ホーム

URL:


Skip to content

Navigation Menu

Sign in
Appearance settings

Search code, repositories, users, issues, pull requests...

Provide feedback

We read every piece of feedback, and take your input very seriously.

Saved searches

Use saved searches to filter your results more quickly

Sign up
Appearance settings

AIX doesn't have tm_gmtoff#876

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 ourterms of service andprivacy statement. We’ll occasionally send you account related emails.

Already on GitHub?Sign in to your account

Merged
eddelbuettel merged 1 commit intoRcppCore:masterfromaixoss:master
Jul 9, 2018
Merged

Conversation

@ayappanec
Copy link
Contributor

AIX doesn't have tm_gmtoff due to which the compilation is failing. This PR will fix this issue.

@codecov-io
Copy link

Codecov Report

Merging#876 intomaster willnot change coverage.
The diff coverage isn/a.

Impacted file tree graph

@@           Coverage Diff           @@##           master     #876   +/-   ##=======================================  Coverage   90.09%   90.09%           =======================================  Files          71       71             Lines        3271     3271           =======================================  Hits         2947     2947             Misses        324      324
Impacted FilesCoverage Δ
src/Date.cpp69.4% <ø> (ø)⬆️

Continue to review full report at Codecov.

Legend -Click here to learn more
Δ = absolute <relative> (impact),ø = not affected,? = missing data
Powered byCodecov. Last updatef3c5a34...efcd91e. Read thecomment docs.

@eddelbuettel
Copy link
Member

eddelbuettel commentedJul 9, 2018
edited
Loading

Hm, we have not had any AIX reports in some time. I wonder why that did not come up before. I see two other files with five total uses for#ifdef _AIX so this will probably not cause harm -- but still strange we did not need it before.

@ayappanec
Copy link
ContributorAuthor

Not sure. I am just trying R in AIX and i found this issue.

@eddelbuetteleddelbuettel merged commite5591c5 intoRcppCore:masterJul 9, 2018
Sign up for freeto join this conversation on GitHub. Already have an account?Sign in to comment

Reviewers

No reviews

Assignees

No one assigned

Labels

None yet

Projects

None yet

Milestone

No milestone

Development

Successfully merging this pull request may close these issues.

3 participants

@ayappanec@codecov-io@eddelbuettel

[8]ページ先頭

©2009-2025 Movatter.jp