Bug 18706 - Создает pid-файл в /
Summary: Создает pid-файл в /
Status: CLOSED FIXED
Alias: None
Product: Sisyphus
Classification: Development
Component: alterator-browser-qt (show other bugs)
Version: unstable
Hardware: all Linux
: P2 blocker
Assignee: Sergey V Turchin
QA Contact: qa-sisyphus
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2009-01-31 11:56 MSK by Alexei V. Mezin
Modified: 2009-02-14 00:48 MSK (History)
5 users (show)

See Also:


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Alexei V. Mezin 2009-01-31 11:56:14 MSK
Имеется 
alterator-standalone-6.1-alt1

После запуска acc прям в корне диска появляется вот это:

[root@threebears /]# ll /a*
-rw-r--r-- 1 root root 5 Янв 31 11:52 /alterator-browser-qt-0.pid

Очень некультурно.
Comment 1 Alexei V. Mezin 2009-01-31 12:14:43 MSK
Вдогонку:

происходит это только при вызове acc от пользователя. В результате, при работе с livecd запуск acc от пользователя невозможен вообще.
Comment 2 inger@altlinux.org 2009-02-02 10:38:00 MSK
pid-файлы создаёт alterator-browser-qt
Comment 3 Andrey Rahmatullin 2009-02-02 10:42:10 MSK
Да, и это дуп #18227, но DUPLICATE на другой бранч вроде ставить нехорошо.
Comment 4 inger@altlinux.org 2009-02-07 10:13:10 MSK
Из-за этого не работает центр управления в acc.
Браузер пытается воспользоваться переменной TMPDIR, а она видимо сбрасывается consolehelper.
Comment 5 inger@altlinux.org 2009-02-07 10:17:45 MSK
(In reply to comment #4)
> Из-за этого не работает центр управления в acc.
Не правильно написал: не работает в livecd где read only /.
Comment 6 Dmitriy Khanzhin 2009-02-08 17:21:39 MSK
Как я понял, он запрашивает TMPDIR и если там пусто, задает ее как /tmp,
но потом это значение не использует.
Мож сделать хотя бы так, а?

--- alterator-browser-qt/main.cc.orig        2009-02-08 15:19:27 +0300
+++ alterator-browser-qt/main.cc     2009-02-08 17:06:18 +0300
@@ -14,7 +14,7 @@
     QString tmpdir(getenv("TMPDIR"));
     if( tmpdir.isEmpty() )
        tmpdir = "/tmp";
-    QString pidfile_path = QString("%1/alterator-browser-qt-%2.pid").arg(getenv("TMPDIR")).arg(getuid());
+    QString pidfile_path = QString("%1/alterator-browser-qt-%2.pid").arg(tmpdir).arg(getuid());
     { // check browser is running
        QFile pidfile(pidfile_path);
        if( pidfile.open(QIODevice::ReadOnly) )
Comment 7 Sergey V Turchin 2009-02-09 15:31:00 MSK
(В ответ на комментарий №6)
> Мож сделать хотя бы так, а?
Да. Именно так и задумывалось.
alterator-browser-qt-2.10.4-alt2
Comment 8 Zerg 2009-02-14 00:48:17 MSK
Забыл отфиксить