Bug 12560 - iso9660 extfs compat fix for newer isoinfo
Summary: iso9660 extfs compat fix for newer isoinfo
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: mc (show other bugs)
Version: unstable
Hardware: all Linux
: P2 enhancement
Assignee: Sergey Y. Afonin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2007-08-14 12:42 MSD by Michael Shigorin
Modified: 2010-12-07 17:25 MSK (History)
1 user (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Michael Shigorin 2007-08-14 12:42:47 MSD
[11:19:51] <Hottab> Привет.
Ты еще командуешь iso9660 для mc-шника, или уже другой кто?
[11:20:02] <gvy> ну... типа да
[11:20:18] <Hottab> багфиксы принимаешь? ;)
[11:24:16] <gvy> угу, в багзиллу только лучше :)
[11:25:37] <Hottab> Ну, примерно так:
[11:25:46] <Hottab> 
--- iso9660-orig        2007-08-13 14:15:10.000000000 +0300
+++ iso9660     2007-08-13 19:06:53.000000000 +0300
@@ -42,7 +42,7 @@
  # Pattern to match 8 first fields.
  rx = "[^     ]+[     ]+";
  rx = "^" rx rx rx rx rx rx rx rx;
-  irx = "^\\[ *[0-9]* *[0-9]+\\]  ";
+  irx = "^\\[ *[0-9]* *[0-9]+\\][ ]+";
}
/^$/ { next }
/^d---------/ { next }
@@ -69,7 +69,7 @@
       $ISOINFO -i "$1" -x "/$2" > "$3"
}

-LC_ALL=C
+#LC_ALL=C

cmd="$1"
shift

[11:27:13] <Hottab> ПрОблем в следующем: с последним isoinfo оно не вырезает
иноды, и рубит нахрен локаль.
С таким патчем трохи лучше, но симлинки все одно выводит с инодами.
[11:38:48] <gvy> мгм.  авторам isoinfo надо намекнуть, что вообще-то его иногда
парсят... или они знают и мстят?..
[11:40:17] <Hottab> Эт уже не ко мне вопрос.
Но.... привязываться к количеству пробелов.... как-то оно.... не дуже...
[11:40:35] <gvy> угу
[11:41:23] <gvy> собсно дело в том, что я могу предложить в альтовский пакет, а
с апстримом довольно сложно -- Паша Роскин отказался тащить mc пару-тройку лет
назад, про другого $MAINTAINER пока не слышал
Comment 1 Andrey Rahmatullin 2009-10-09 19:41:18 MSD
Шотут.
Comment 2 Andrey Rahmatullin 2009-10-09 19:42:25 MSD
Сейчас там
  irx = "^\\[ *-?[0-9]* *[0-9]+\\] +";
Я так понимаю, это равносильно всему, что предложено в данной баге, или ещё какие-то варианты будут?
Comment 3 Sergey Y. Afonin 2010-12-07 17:25:18 MSK
(In reply to comment #2)

> Сейчас там
>   irx = "^\\[ *-?[0-9]* *[0-9]+\\] +";
> Я так понимаю, это равносильно всему, что предложено в данной баге, или ещё
> какие-то варианты будут?

Мне тоже кажется, что равносильно. Закрываю. Если что, reopen никто не отменял...