Bug 24635 - poedit seems to hang after save/validate until movement/refresh
Summary: poedit seems to hang after save/validate until movement/refresh
Status: RESOLVED FIXED
Alias: None
Product: Mageia
Classification: Unclassified
Component: RPM Packages (show other bugs)
Version: 7
Hardware: All Linux
Priority: Normal normal
Target Milestone: ---
Assignee: David GEIGER
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2019-04-07 13:26 CEST by aguador
Modified: 2020-04-22 09:59 CEST (History)
2 users (show)

See Also:
Source RPM: poedit-2.2.1-1.mga7.src.rpm
CVE:
Status comment:


Attachments
strace of poedit (158.67 KB, text/plain)
2019-04-10 16:37 CEST, aguador
Details
Screenshot poedit 2.2.2 error (15.29 KB, image/jpeg)
2019-05-17 08:27 CEST, aguador
Details

Description aguador 2019-04-07 13:26:14 CEST
When saving a file in poedit, an error notification reports that the permissions could not be changed on the temporary file. Upon closing the pop-up, poedit is then unresponsive and must be closed.

No loss of data in the translation has been noted.

To reproduce: open a .po file to translate, make a change, and save it. The error will pop up and the program be unresponsive after closing the notification.

This is running poedit on an 86_64 machine with the Enlightenment DE.

This is not a bug I have seen reported upstream.
Comment 1 David GEIGER 2019-04-07 13:37:40 CEST
Just tested here with some .po files and I can't reproduce your issue.

CC: (none) => geiger.david68210

Comment 2 Marja Van Waes 2019-04-09 10:30:53 CEST
Assigning to our registered  maintainer.

CC: (none) => marja11
Assignee: bugsquad => geiger.david68210

Comment 3 aguador 2019-04-09 11:37:13 CEST
(In reply to David GEIGER from comment #1)
> Just tested here with some .po files and I can't reproduce your issue.

Perhaps it was related to the particular po file I was sent (large for the few .po files I've worked with: 356K), but that file worked in the 2.0 version in Mageia 6 without throwing the error. Will check this as soon as I can as my Cauldron laptop is now down with a dead fan.
Comment 4 aguador 2019-04-10 16:37:47 CEST
Created attachment 10916 [details]
strace of poedit

Have not tested this with a couple of files and all produce the error. Also changed the theme from Vertex Dark to Adwaita (light), and got the same error.
Comment 5 aguador 2019-04-10 16:38:43 CEST
(In reply to aguador from comment #4)

-Typo: have now tested with a couple of different files.
Comment 6 aguador 2019-05-17 08:27:13 CEST
Created attachment 10999 [details]
Screenshot poedit 2.2.2 error

Poedit 2.2.2 saves successfully, but still reports permissions error.
Comment 7 David GEIGER 2019-05-17 13:36:45 CEST
Can you open a new bug upstream to see what is going wrong, please?
Comment 8 aguador 2019-05-20 20:07:04 CEST
OK, thanks David, just received a response from upstream where Vlaclav (less obnoxious than my last interaction) insists it is an ENV or messed up permissions problem.

I did try running poedit-2.2.2 under Icewm as I was wondering if E might have been setting some ENV variable. However, it hung up with Icewm as well. I have also checked ENV variables (printenv) in E and see nothing to suggest a problem there.

Oh, and having seen the exchanges in the dev ml about SELINUX, I also attempted with kernel 4.20 that I still have kicking around. No luck.

Any suggestions for what more I can do to determine the problem?

Roy
Comment 9 katnatek 2019-05-20 21:56:01 CEST
(In reply to aguador from comment #8)
> Any suggestions for what more I can do to determine the problem?
> 
> Roy

1. Run poedit from terminal , see if produce some messages
2. Install if you can the xfce desktop and
2a Test in xfce
2b Test again in E , just in case is a missing require issue
Comment 10 aguador 2019-05-26 14:57:09 CEST
(In reply to katnatek from comment #9)
> (In reply to aguador from comment #8)
> 1. Run poedit from terminal , see if produce some messages
> 2. Install if you can the xfce desktop and
> 2a Test in xfce
> 2b Test again in E , just in case is a missing require issue
Thanks for the tips. There is no additional information from running from the terminal (already tried). I am hesitant to install XFCE as it will drag a lot of dependencies I then need to clean up. I did check the dependencies in the SRPM and there are basically other than gettext and no recommendeds. I have also checked tmp directory permissions (where I assume  which are OK).

My Cauldron (but not Mga6) system has a SECSHELL variable set either by Mageia or E, but even eliminating the variable does not change the behaviour.

I should add that the problem is less serious than I thought.

First, if the 2.2.3 does produce the error message, it prevents further editing. However, the file is saved and can simply be opened again to resume translating.

Second, the files I have been opening open at the beginning, but the default setting is for strings with problems to be displayed first. Eliminating this option before saving results in the error to be show, but editing can be resumed normally after acknowledging the error.

Summary: poedit reports a temporary file permissions error and locks up when saving => poedit reports a temporary file permissions error and prevents editing of current file

Comment 11 aguador 2019-05-26 15:05:37 CEST
Oops, I changed the text of the bug before finishing the last message. The upshot of what I have seen seems to be:

a) A permissions problem with the tmp file on my system that I cannot find. The error is not reported if poedit is run as root.

I have not been able to track the tmp file location. Does anyone know where it is written?

b) The impossibility of continuing editing after save if the view setting is set to show strings with errors first, something that occurs even when poedit is run as root. As no permissions error is shown, I assume this is a separate problem.

Is anyone else able to replicate this?
aguador 2019-05-26 17:26:14 CEST

Summary: poedit reports a temporary file permissions error and prevents editing of current file => poedit seems to hang after save/validate until movement/refresh

Comment 12 aguador 2019-05-26 17:28:32 CEST
OK, bug report significantly modified to reflect further testing. More details in upstream bug report:

https://github.com/vslavik/poedit/issues/578
Comment 13 aguador 2020-04-22 09:59:58 CEST
Resolved in later releases.

Resolution: (none) => FIXED
Status: NEW => RESOLVED


Note You need to log in before you can comment on or make changes to this bug.