и его можно сразу в спеке применять?

да

единственное, у него название будет не очень

но

можно в конце сделать

#/foo-bar.patch

и он должен скачаться под таким именем

что-то он не применяется

покажи лог

https://paste.fedoraproject.org/401047/14702925

а с чего ты взял, что он применится без конфликтов ?)

в теории можешь попробовать git 3way merge

BuildRequires: git-core

%autosetup -Sgit

Ну он по идее отменяет одно изменение. И глазами смотрю - не видно чтобы там конфликт был. https://github.com/vgough/encfs/commit/87ee4b7d99781630c467df265dbe0b66b98de3b5?diff=unified

хотя... номера строк другие

ладно, сделаю патч вручную

[ignatenk@ignatenko encfs]$ git checkout v1.8.1 Note: checking out 'v1.8.1'. You are in 'detached HEAD' state. You can look around, make experimental changes and commit them, and you can discard any commits you make in this state without impacting any branches by performing another checkout. If you want to create a new branch to retain commits you create, you may do so (now or later) by using -b with the checkout command again. Example: git checkout -b <new-branch-name> HEAD is now at 843a4ca... Merge pull request #69 from rfjakob/next [ignatenk@ignatenko encfs]$ git cherry-pick 87ee4b7d99781630c467df265dbe0b66b98de3b5 error: could not apply 87ee4b7... Revert "Remove "-o default_permissions" unless needed." hint: after resolving the conflicts, mark the corrected paths hint: with 'git add <paths>' or 'git rm <paths>' hint: and commit the result with 'git commit'

@Vascom2 а зачем ты пытаешься бэкпортить фикс, который не нужен?

я в том смысле, что в 1.8.1 ещё не сломали эту штуку

сломали

а, вижу

вот багрепорт https://bugzilla.redhat.com/show_bug.cgi?id=1363723

0001-Revert-Remove-o-default_permissions-unless-needed.patch: https://paste.fedoraproject.org/401050/14702936

@Vascom2 ^^

спасибо, я уже сделал

спасибо, я уже сделал

как именно ты сделал?

твой патч некорректен

чем он некорректен?

Я правда неправильно положил его...

чем он некорректен?

посмотри на мой патч и на свой

у тебя аргументы в другой вункции добавляются

В той же if (useDefaultFlags)

функция другая

точнее нет

оно раньше должно добавляться

да всё то же самое

суть в том, что нужно было все отревертить коммит, а не пытаться забекпортить новый

куда раньше-то?

с чего это?

это же патч

ну у тебя было много изменений после оригинального коммита

поэтому реверт того коммита в v1.8.1 и в мастере - абсолютно разные патчи

поэтому у тебя и не накладывается

т.е. что нужно было сделать

$ git clone ... $ cd ... $ git checkout v1.8.1 $ git revert ... $ git format-patch -1

я взял изменения из мастера и применил

я взял изменения из мастера и применил

нет, ты ещё раз их заребейзил

изменение из мастера напрямую не применяется

что это?

я вручную взял

я вручную взял

и получил не совсем то, что должен сделать git-revert

https://paste.fedoraproject.org/401053/02945791

ну не обязателен же тут гит-реверт

или обязателен?

у тебя результат другой получится

ри

смотри

в мастере они там что-то поменяли с аргументами. потом сделали git-revert и у них получились конфликты с новым подходом к парсингу аргументов, они их решили. но в v1.8.1 там нет таких конфликтов, т.к. это другой код

но в твоём патче всё то же самое

не-а

if (!out->isThreaded) PUSHARG("-s"); + if (useDefaultFlags) { + PUSHARG("-o"); + PUSHARG("use_ino"); + PUSHARG("-o"); + PUSHARG("default_permissions"); + } + // we should have at least 2 arguments left over - the source directory and

373 строка

а в твоём патче, это добавляется где-то в 400 строке

думаешь от расположения этого if что-то изменится?

после каких-то if()

ну там не только иф по идее

я код детально не смотрел

только

тем более в 1.8.1 этот if именно на 408 строке что ли

399

а не 373

373

https://paste.fedoraproject.org/401050/14702936/

@@ -373,6 +373,13 @@ static bool processArgs(int argc, char *argv[],

399 https://github.com/vgough/encfs/blob/843a4ca5be13b78b947e34c8fe491b9ff12ed4e2/encfs/main.cpp

это в релизе 1.8.1

посмотри на оригинальный патч, который ревертят

оригинальный был до этого

там же надо просто условие убрать внутри данного if()