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

[stable6] Update SabreDAV to 1.7.11 #80

Closed
wants to merge 1 commit into from

Conversation

PVince81
Copy link
Contributor

@PVince81 PVince81 commented Mar 3, 2014

⛔ NOMERGE ⛔

DON'T merge before 6.0.2 is released.

Backport of c77c4dd to stable6

@DeepDiver1975 as discussed.

Are the autoloader changes necessary for that update ?

Backport of c77c4dd to stable6
@georgehrke
Copy link
Contributor

@DeepDiver1975 Does cleaning up the 3rdparty "mess" also include removing VObject from core's 3rdparty folder, as it is not needed by core?

@georgehrke
Copy link
Contributor

(a bit off topic, but related to keeping SabreDav up to date)

@DeepDiver1975
Copy link
Contributor

@PVince81 what shall we do here? THX

@DeepDiver1975
Copy link
Contributor

Does cleaning up the 3rdparty "mess" also include removing VObject from core's 3rdparty folder, as it is not needed by core?

we will keep sabre in 3rdparty including it's dependencies like vobject

@PVince81
Copy link
Contributor Author

I don't really remember why we wanted that backport, possibly an important bugfix (CC @LukasReschke)

@LukasReschke
Copy link
Contributor

c77c4dd#diff-0d9497d33b1fc973114332b929a080fcR535

It's a bugfix. It may fail otherwhise when using PHP-FPM due to a PHP bug.

@LukasReschke
Copy link
Contributor

"fail" = ownCloud may not run properly at all.

@PVince81
Copy link
Contributor Author

@DeepDiver1975 I guess this is obsolete, close ?

@LukasReschke
Copy link
Contributor

7 has this. 6 can live without fine to close for me.

@DeepDiver1975 DeepDiver1975 deleted the stable6-backportsabreupdate branch February 17, 2015 11:28
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