Skip to content

[SR-4905] Store cookies in XDG_DATA_HOME instead of XDG_CONFIG_HOME #1000

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

Merged
merged 1 commit into from
May 23, 2017

Conversation

mamabusi
Copy link
Contributor

No description provided.

@pushkarnk
Copy link
Member

1000th pull request !

@mamabusi mamabusi force-pushed the storeCookieInDataHome branch from 4d79715 to b491d22 Compare May 22, 2017 11:09
@pushkarnk
Copy link
Member

@swift-ci test

@pushkarnk
Copy link
Member

@swift-ci please test

@mamabusi mamabusi force-pushed the storeCookieInDataHome branch from b491d22 to b1ccf49 Compare May 22, 2017 17:31
@mamabusi
Copy link
Contributor Author

@pushkarnk Could you please run the CI Build again. I have pushed in the necessary changes to fix the test-failures.

@parkera
Copy link
Contributor

parkera commented May 22, 2017

@swift-ci please test

1 similar comment
@pushkarnk
Copy link
Member

@swift-ci please test

@mamabusi
Copy link
Contributor Author

mamabusi commented May 23, 2017

@pushkarnk @parkera The CI test-failure seen is because of a bug in Cookie Storage where it fails to create non-existing parent directories. A fix for this is already available in the open PR: #1003

@pushkarnk
Copy link
Member

We have #1003 merged now. Lets try to merge this one as well.

@swift-ci test and merge

@pushkarnk
Copy link
Member

@swift-ci test and merge

@swift-ci swift-ci merged commit 667dde1 into swiftlang:master May 23, 2017
@mamabusi mamabusi changed the title Store cookies in XDG_DATA_HOME instead of XDG_CONFIG_HOME [SR-4905] Store cookies in XDG_DATA_HOME instead of XDG_CONFIG_HOME May 23, 2017
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.

4 participants