metaclass: (Default)
metaclass ([personal profile] metaclass) wrote2012-11-10 09:41 pm

Меркуриал

Одной вещи в меркуриале я в принципе понять не могу.
Я меняю файл А, делаю commit и push, потом я меняю дома файл Б и делаю commit, а на попытку push меня посылают фпень делать pull/merge/commit.
Вопрос - почему бы независимые изменения в разных папках не мержить автоматом, как это сделано в svn?

PS: А теперь рассмотрим, как это положено делать:
1) дерево исходников представляет собой граф, узлы которого - каталоги и блобы с файлами, а связи - связь между каталогом и расположенными в нем подкаталогами и файлами.
2) пользователь 1 меняет файл с путем A
3) пользователь 2 меняет другой файл с путем B.
4) Друг другу они в принципе никак не мешают - структура графа не меняется, меняются два независимых узла.
5) операции коммутируют - т.е. вне зависимости от порядка применения операций, результат будет одним и тем же.
6) для коммутирующих операций merge должен делаться, очевидно, автоматически.

[identity profile] kaa-mmf.livejournal.com 2012-11-10 06:57 pm (UTC)(link)
м-м-м, может я чейта не понимаю, но если последний пуш был от тебя, то никаких доп. приседаний при след. пуше не надо

[identity profile] metaclass.livejournal.com 2012-11-10 07:10 pm (UTC)(link)
Пуш от меня, но с разных рабочих мест.

(no subject)

[identity profile] kaa-mmf.livejournal.com - 2012-11-10 22:39 (UTC) - Expand

[identity profile] avnik.livejournal.com 2012-11-10 07:04 pm (UTC)(link)
Я могу объяснить на пальцах -- почему гит тебя пошлет мержить. А тут -- хрен его знает. Корни проблемы те же я подозреваю

[identity profile] metaclass.livejournal.com 2012-11-10 07:11 pm (UTC)(link)
git тоже пошлет? Как они задолбут.

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 19:21 (UTC) - Expand

(no subject)

[identity profile] avnik.livejournal.com - 2012-11-10 19:22 (UTC) - Expand

(no subject)

[identity profile] nivanych.livejournal.com - 2012-11-10 19:46 (UTC) - Expand

(no subject)

[identity profile] metaclass.livejournal.com - 2012-11-10 19:49 (UTC) - Expand

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 20:01 (UTC) - Expand

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 20:22 (UTC) - Expand

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 20:39 (UTC) - Expand

(no subject)

[identity profile] cottidianus.livejournal.com - 2012-11-11 00:42 (UTC) - Expand

(no subject)

[identity profile] avnik.livejournal.com - 2012-11-10 20:28 (UTC) - Expand

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 20:31 (UTC) - Expand

(no subject)

[identity profile] avnik.livejournal.com - 2012-11-10 20:41 (UTC) - Expand

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 20:50 (UTC) - Expand

(no subject)

[identity profile] avnik.livejournal.com - 2012-11-10 20:59 (UTC) - Expand

(no subject)

[identity profile] metaclass.livejournal.com - 2012-11-11 07:50 (UTC) - Expand

(no subject)

[identity profile] gds.livejournal.com - 2012-11-11 07:57 (UTC) - Expand

(no subject)

[identity profile] vaddimka.livejournal.com - 2012-11-11 07:44 (UTC) - Expand

[identity profile] juan-gandhi.livejournal.com 2012-11-11 05:27 am (UTC)(link)
Можно и мне на пальцах?

А то я коммит-пуш только так делаю; а пулл-ребейс по настроению.

[identity profile] kiryl.livejournal.com 2012-11-10 08:21 pm (UTC)(link)
> PS: А теперь рассмотрим, как это положено делать:

Ты упускаешь важную часть графа. Про mercurial не скажу, в git оно выглядит так: есть четыре вида объектов: blob, tree, commit, tag. Все объекты адресуются по содержимому -- хэш от содержимого.

Что содежит blob понятно. tree -- список ссылок на blob'ы и другие деревья. Таким образом строится иерархия. commit -- ссылка на корневое дерево проекта, commit message + нуль или более ссылок на parent commit (больше 1 для merge commit). tag -- ссылка любой объект, чаще всего commit.

Из вышесказанного очевидно, что изменение одного blob'а повлечёт к изменению всех объектов выше по стеку: blob -> tree -> commit -> tag.

Т.е. операции не коммутирующие. И это сделано намеряно. Для distributed scm критически важно лёгкое отслеживание целостности, дабы избежать code injection и прочие проблемы с безопастностью.

[identity profile] avnik.livejournal.com 2012-11-10 08:35 pm (UTC)(link)
Ребе упускает из вида, что у нас есть еще C -- сервер.

4/5 - ломает, ломает порядок коммитов: будет ломать ребейс. Будет появляться лишний merge сщььше на C которого нет ни на A ни на B

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-10 20:36 (UTC) - Expand

[identity profile] voidex.livejournal.com 2012-11-10 08:39 pm (UTC)(link)
Не должен. Если A' + B работает, и A + B' работает, не значит, что A' + B' будет тоже работать, пусть программист об этом знает и ручается.
Смержит он сам, достаточно pull+push.

[identity profile] molnij.livejournal.com 2012-11-11 05:31 am (UTC)(link)
+1

[identity profile] max630.livejournal.com 2012-11-10 09:27 pm (UTC)(link)
бгг

Обычно это делается так: пользователь A заводит merge request, потом пользователь B делает review изменений и заставляет A исправить ошибки. Потом специальный человек, когда у него дойдут руки (примерно раз в несколько дней), собирает эти реквесты и мержит (или ребейзит) их в мастер, при необходимости дёргая A чтобы он поправил конфликт.

А вы, ну я не знаю, смиритесь наверное. Или наколхозьте автомержилку на сервере.

[identity profile] djuffin.livejournal.com 2012-11-10 09:46 pm (UTC)(link)
Чтобы программист знал, что были изменения и деклорация метода который он вызвает была удален из совершенно независимого файла.

[identity profile] anatoly borodin (from livejournal.com) 2012-11-13 05:40 pm (UTC)(link)
Дадад. «Мы тут только в нашей папке меняли» — а ничего, что при этом сломали 5 модулей в других папках?

Свнщики, мать их так…
Edited 2012-11-13 17:41 (UTC)

[identity profile] berezovsky.livejournal.com 2012-11-11 02:34 am (UTC)(link)
Ну это ты знаешь, что они независимые. А он не знает.
Пришёл домой, сделал пул, вспомнил, что во второй папке написал херню, поправил и запушил всё сразу.
Я так понимаю.

[identity profile] gds.livejournal.com 2012-11-11 02:43 am (UTC)(link)
> а на попытку push меня посылают делать pull/merge/commit.

не обязательно. hg push -f не посылает. Очень полезно в случаях, когда мержить должен не тот, кто пушит.
Кроме того, вроде где-то видел хрень (то ли настройка, то ли расширение), которое то ли команду добавляет, то ли опцию к push, вызывающую автоматический pull + merge + commit + push, или же rebase, не помню. (кстати, "hg rebase" таки существует.)

Остальное сказали.

Ну и да, merge сделается почти автоматически -- без конфликтов.

(no subject)

[identity profile] gds.livejournal.com - 2012-11-11 06:43 (UTC) - Expand

[identity profile] vp.livejournal.com 2012-11-11 06:59 am (UTC)(link)
Почитал комменты и ничего не понял, что делать.

[identity profile] jek-hor.livejournal.com 2012-11-11 07:32 am (UTC)(link)
Мержить самостоятельно. Как выше правильно написали, только человек может гарантировать целостность кода, даже если он в разных директориях менялся в каждом коммите.

(no subject)

[identity profile] jek-hor.livejournal.com - 2012-11-18 08:44 (UTC) - Expand

(no subject)

[identity profile] metaclass.livejournal.com - 2012-11-18 08:51 (UTC) - Expand

(no subject)

[identity profile] jek-hor.livejournal.com - 2012-11-18 08:55 (UTC) - Expand

[identity profile] metaclass.livejournal.com 2012-11-11 08:00 am (UTC)(link)
расширение hg fetch
оно при неконфликтующих коммитах автоматический мердж будет делать.

(no subject)

[identity profile] max630.livejournal.com - 2012-11-11 17:28 (UTC) - Expand

(no subject)

[identity profile] dmzlj.livejournal.com - 2012-11-11 17:28 (UTC) - Expand

(no subject)

[identity profile] vp.livejournal.com - 2012-11-11 18:57 (UTC) - Expand

(no subject)

[identity profile] kiryl.livejournal.com - 2012-11-11 19:44 (UTC) - Expand

(no subject)

[identity profile] dmzlj.livejournal.com - 2012-11-12 04:09 (UTC) - Expand

(no subject)

[identity profile] thinker8086.livejournal.com - 2012-11-13 00:37 (UTC) - Expand

[identity profile] x-den.livejournal.com 2012-11-11 07:43 am (UTC)(link)
если проблема в лишних движениях, то поможет hg fetch (FetchExtension), он pull/merge/commit делает автоматически, и если нет конфликтов, то вмешательства в это дело не требуется.

оопс, его уже выше успели предложить.
Edited 2012-11-11 07:44 (UTC)

[identity profile] vit-r.livejournal.com 2012-11-11 10:18 am (UTC)(link)
Друг другу они в принципе никак не мешают

Это не факт.

[identity profile] zamotivator.livejournal.com 2012-11-11 10:22 am (UTC)(link)
Плохая идея. Изменения могут быть в разных файлах, а результатом автоматического мёржа - неработающая программа.

Правильный подход либо mercurial - держать два head, либо git - запрещать push с outdated head

[identity profile] insanegigolo.livejournal.com 2012-11-11 10:45 am (UTC)(link)
разве за неработающую программу не должна отвечать билд система, а не система контроля версий?

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 10:47 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 10:52 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 11:07 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 11:14 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 11:20 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 11:29 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 11:32 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 11:34 (UTC) - Expand

(no subject)

[identity profile] thinker8086.livejournal.com - 2012-11-13 00:39 (UTC) - Expand

(no subject)

[identity profile] metaclass.livejournal.com - 2012-11-11 10:56 (UTC) - Expand

(no subject)

[identity profile] zamotivator.livejournal.com - 2012-11-11 10:57 (UTC) - Expand

(no subject)

[identity profile] gds.livejournal.com - 2012-11-11 11:06 (UTC) - Expand

[identity profile] levgem.livejournal.com 2012-11-11 10:55 am (UTC)(link)
Это нормально.

Изменения в коммите 1 могут быть идеологически конфликтными с изменениями в коммите 2, даже если они совершенно разные вещи затрагивают.
Поэтому перед пушем коммита 2 надо сделать проверку.

[identity profile] dmzlj.livejournal.com 2012-11-11 04:43 pm (UTC)(link)
а пулл сначала на второе рабочее место не судьба сделать? локальные ревизии то разъехались, очевидно

[identity profile] dmzlj.livejournal.com 2012-11-11 04:48 pm (UTC)(link)
вы тут точно не обкурились все? есть стандартный workflow, который делает то, что нужно, и покрывает 90% случаев.

pull - update - commit - push. с любого рабочего места. всё. какие еще файлы? какая коммутация-мастурбация? вы чего тут?

[identity profile] berezovsky.livejournal.com 2012-11-11 05:05 pm (UTC)(link)
Ситуации разные бывают. Например, сидишь в аэропорту через узкий канал какого-нибудь опсоса. Зачем тянуть пулом кучу всякого дерьмища и тратить трафик? Вот метакласс и хочет запушить один файлик и всё.

(no subject)

[identity profile] dmzlj.livejournal.com - 2012-11-11 17:09 (UTC) - Expand

(no subject)

[identity profile] berezovsky.livejournal.com - 2012-11-11 17:13 (UTC) - Expand

(no subject)

[identity profile] dmzlj.livejournal.com - 2012-11-11 17:13 (UTC) - Expand

(no subject)

[identity profile] berezovsky.livejournal.com - 2012-11-11 17:19 (UTC) - Expand

(no subject)

[identity profile] ykaliuta.livejournal.com - 2012-11-11 17:21 (UTC) - Expand

(no subject)

[identity profile] dmzlj.livejournal.com - 2012-11-11 17:23 (UTC) - Expand

(no subject)

[identity profile] metaclass.livejournal.com - 2012-11-11 17:34 (UTC) - Expand

(no subject)

[identity profile] metaclass.livejournal.com - 2012-11-11 17:24 (UTC) - Expand

(no subject)

[identity profile] dmzlj.livejournal.com - 2012-11-11 17:10 (UTC) - Expand

[identity profile] http://users.livejournal.com/_slw/ 2012-11-11 05:40 pm (UTC)(link)
ну так и используй svn

[identity profile] nealar.livejournal.com 2012-11-11 06:49 pm (UTC)(link)
+1