Welcome! Log In Create A New Profile

Advanced

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin
October 16, 2012 02:36PM
Hello!

On Tue, Oct 16, 2012 at 10:04:08PM +0400, Михаил Монашёв wrote:

> Здравствуйте, Maxim.
>
> >> А если потребуется поменять atime, то:
> >> - if (ngx_set_file_time(NULL, fd, ctx->mtime) != NGX_OK) {
> >> + if (ngx_set_file_time(NULL, fd, ctx->atime) != NGX_OK) {
> >>
> >> ?
> >>
> >> А то я тут посмотрел код скриптов, работающих с вебдавом, чтоб туда
> >> идейку про перл в комменты записать. А там мы оказывается atime
> >> меняем, а не mtime.
>
> > Это cut-n-paste ошибка в виндовой части кода, которую я поленился
> > проверить и недоправил. Там должно быть date, исправленный патч
> > прилагается.
>
> >> И, кстати, Игорь прав. ;-) Глядя на твой патч возникает желание
> >> плодить методы под каждый чих, что неправильно.
>
> > С одной стороны да, с другой - иногда надо хотя бы пытаться
> > предлагать какую-то альтернативу тому ужасу, что стандартизировал
> > Microsoft. Особенно с учётом того, сколько методов они сами
> > наплодили как в стандарте на webdav, так и в дополнение к нему.
> > :)
>
> Ну тогда уж предлагать полноценную альтернативу. А то чем это
> отличается от их изысканий. Конечно, TOUCH понятен, привычен и
> наверное востребован, но это частность. Этот TOUCH станет не нужен,
> если реализовать нормальную альтернативу PROPPATCH без вывихов

Есть мнение, что TOUCH, возможно, заслуживает своего собственного
метода. Именно потому, что - понятен и привычен, и местами
востребован. И что отдельно немаловажно - прост в реализации.

> сознания. Да и там много чего можно было б реализовать правильного,
> например позаимствовать из мемкешеда cas...

CAS, кстати, в HTTP/1.1 есть - заголовки If-* позволяют всякого.
Правда, в реальной жизни это не совсем работает, или даже совсем
не работает, особенно если говорить про nginx.

> И, кстати, у nginx-а есть возможность заказывать музыку, предлагая
> свои реализации альтернатив. ИМХО.

Ну вот я TOUCH нарисовал, если приживётся - хорошо, не приживётся -
значит не судьба. :)

--
Maxim Dounin
http://nginx.com/support.html

_______________________________________________
nginx-ru mailing list
nginx-ru@nginx.org
http://mailman.nginx.org/mailman/listinfo/nginx-ru
Subject Author Posted

[PATCH] implemented hardlink option in DAV module

arut October 15, 2012 07:42AM

Re: [PATCH] implemented hardlink option in DAV module

kav October 15, 2012 07:48AM

Re: [PATCH] implemented hardlink option in DAV module

arut October 15, 2012 07:58AM

Re: [PATCH] implemented hardlink option in DAV module

kav October 15, 2012 08:04AM

Re: [PATCH] implemented hardlink option in DAV module

arut October 15, 2012 08:10AM

Re: [PATCH] implemented hardlink option in DAV module

kav October 15, 2012 08:24AM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 15, 2012 11:34AM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 15, 2012 03:32PM

Re: [PATCH] implemented hardlink option in DAV module

Gena Makhomed October 15, 2012 03:58PM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 15, 2012 04:16PM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 16, 2012 06:18AM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 16, 2012 01:12PM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 16, 2012 01:32PM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 16, 2012 01:50PM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 16, 2012 02:06PM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 16, 2012 02:36PM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 16, 2012 03:04PM

Re: [PATCH] implemented hardlink option in DAV module

kav October 17, 2012 01:34AM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 17, 2012 05:28AM

Re: [PATCH] implemented hardlink option in DAV module

kav October 17, 2012 06:02AM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв January 14, 2014 12:34PM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin January 15, 2014 11:40AM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв January 15, 2014 01:18PM

Re: [PATCH] implemented hardlink option in DAV module

Валентин Бартенев January 15, 2014 02:12PM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 16, 2012 01:18PM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 16, 2012 01:38PM

Re[2]: [PATCH] implemented hardlink option in DAV module

Михаил Монашёв October 16, 2012 01:54PM

Re: [PATCH] implemented hardlink option in DAV module

Maxim Dounin October 16, 2012 02:06PM

Re: [PATCH] implemented hardlink option in DAV module

arut October 16, 2012 08:44AM

[PATCH] implemented DAV copy hardlinks

arut October 16, 2012 08:54AM

Re: [PATCH] implemented DAV copy hardlinks

Gena Makhomed October 16, 2012 11:42AM

Re[2]: [PATCH] implemented DAV copy hardlinks

Михаил Монашёв October 16, 2012 12:50PM

Re: [PATCH] implemented DAV copy hardlinks

arut October 16, 2012 01:08PM

Re: [PATCH] implemented DAV copy hardlinks

Maxim Dounin October 16, 2012 11:46AM

Re: [PATCH] implemented DAV copy hardlinks

arut October 19, 2012 03:22AM

Re: [PATCH] implemented DAV copy hardlinks

Maxim Dounin October 24, 2012 02:04PM



Sorry, only registered users may post in this forum.

Click here to login

Online Users

Guests: 251
Record Number of Users: 8 on April 13, 2023
Record Number of Guests: 421 on December 02, 2018
Powered by nginx      Powered by FreeBSD      PHP Powered      Powered by MariaDB      ipv6 ready