KDE CI: Applications okular kf5-qt5 FedoraQt5.8 - Build # 86 - Still Unstable!

[CI System]

BUILD UNSTABLE
 Build URL
https://build.kde.org/job/Applications%20okular%20kf5-qt5%20FedoraQt5.8/86/
 Project:
Applications okular kf5-qt5 FedoraQt5.8
 Date of build:
Fri, 20 Oct 2017 01:05:26 +
 Build duration:
13 min and counting
   JUnit Tests
  Name: (root) Failed: 1 test(s), Passed: 15 test(s), Skipped: 0 test(s), Total: 16 test(s)Failed: TestSuite.mainshelltest
   Cobertura Report
  
   Project Coverage Summary
  
   Name
  PackagesFilesClassesLinesConditionalsCobertura Coverage Report88%
(21/24)46%
(124/268)46%
(124/268)29%
(10896/37944)19%
(4957/26208)Coverage Breakdown by Package
Name
   FilesClassesLinesConditionalsdefault80%
(4/5)80%
(4/5)48%
(787/1649)29%
(362/1249)autotests93%
(13/14)93%
(13/14)83%
(1631/1976)38%
(949/2477)conf6%
(1/17)6%
(1/17)7%
(53/805)0%
(1/272)conf.autotests100%
(1/1)100%
(1/1)100%
(17/17)50%
(7/14)core83%
(40/48)83%
(40/48)46%
(4179/9133)34%
(2022/6015)core.script0%
(0/9)0%
(0/9)0%
(0/494)0%
(0/230)generators.comicbook29%
(2/7)29%
(2/7)3%
(10/397)0%
(0/430)generators.dvi2%
(1/46)2%
(1/46)0%
(9/3571)0%
(1/2423)generators.fax17%
(1/6)17%
(1/6)2%
(7/432)0%
(0/585)generators.fictionbook60%
(3/5)60%
(3/5)3%
(16/531)0%
(2/454)generators.kimgio100%
(1/1)100%
(1/1)63%
(41/65)31%
(5/16)generators.kimgio.tests100%
(1/1)100%
(1/1)100%
(49/49)46%
(24/52)generators.mobipocket60%
(3/5)60%
(3/5)11%
(13/115)0%
(0/120)generators.ooo27%
(3/11)27%
(3/11)1%
(14/1089)0%
(0/708)generators.plucker100%
(1/1)100%
(1/1)4%
(4/98)0%
(0/36)generators.plucker.unpluck0%
(0/6)0%
(0/6)0%
(0/1652)0%
(0/876)generators.poppler60%
(3/5)60%
(3/5)44%
(575/1311)35%
(247/697)generators.tiff100%
(1/1)100%
(1/1)4%
(10/226)0%

KDE CI: Applications okular kf5-qt5 FreeBSDQt5.7 - Build # 92 - Fixed!

[CI System]

BUILD SUCCESS
 Build URL
https://build.kde.org/job/Applications%20okular%20kf5-qt5%20FreeBSDQt5.7/92/
 Project:
Applications okular kf5-qt5 FreeBSDQt5.7
 Date of build:
Fri, 20 Oct 2017 01:05:26 +
 Build duration:
12 min and counting
   JUnit Tests
  Name: (root) Failed: 0 test(s), Passed: 15 test(s), Skipped: 0 test(s), Total: 15 test(s)

[okular] [Bug 385468] "Missing \"Version=5\", file '/usr/share/kconf_update/okular.upd' will be skipped."

[Albert Astals Cid]

https://bugs.kde.org/show_bug.cgi?id=385468

--- Comment #5 from Albert Astals Cid  ---
(In reply to Matthew Dawson from comment #4)
> The original reason this was added was that KDE4 update scripts would get
> run by KF5, creating a configuration file.  Then the KDE4 migration would
> not run, resulting in users losing their settings.  This was put in at the
> time to stop that from occurring.
> 
> Ideally, kconf_update would not create an empty configuration file for a
> migration just to note the migration had been run.  However, there are some
> side effects to that change which make it more complicated to implement, and
> thus has not yet happened.
> 
> Without fixing kconf_update, I don't want to back out the change as it still
> has a good reason.  I also like that all the update files now have an
> enforced version header, which will make it easy to update the format in the
> future.  And most configuration file update scripts should now have the
> header if required, so there will likely be more pain to revert it at this
> time.

Ok

> 
> However, you do raise the good point that this requirement is not well
> communicated.  That should definitely be fixed.  Is there any particular
> place you would have expected this information to be present?

the kde-devel mailing list seems like a good place to me to warn kde developers
that a change like this is happening, since otherwise we're not going to see it
easily.

> 
> Also, if the configuration update script is for KDE4 configuration files,
> you can skip putting a Version=5 header at the top.  Instead, you can have
> the fully updated file migrated to the new configuration file location and
> start with a fresh update script for KF5.  Or continue adding to the file. 
> Whatever works best for okular.

For my application there's no such thing as "KDE4" configuration files, they
are just configuration files, and ideally one should be able to go from an old
version (based on kdelibs4, irrelevant) where the okular.upd had not been run
to the new version (based on KF5, irrelevant) and get okular.upd run and then
the config be migrated to the new location.

As I understand you're saying that this is impossible and thus i basically
should just remove the .upd file?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 315552] Allow save annotations directly to original pdf file

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=315552

--- Comment #22 from Nate Graham  ---
Yeah, looks like it. Even though that bugs is older, let's consolidate here
since this bug is clearer and more technical, and doesn't have so much
distracting argument in it.

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 315552] Allow save annotations directly to original pdf file

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=315552

Nate Graham  changed:

   What|Removed |Added

 CC||flying-sh...@web.de

--- Comment #21 from Nate Graham  ---
*** Bug 301774 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 301774] Wish: New option to save embedded annotation upon change/document close

[Nate Graham]

https://bugs.kde.org/show_bug.cgi?id=301774

Nate Graham  changed:

   What|Removed |Added

 Resolution|--- |DUPLICATE
 Status|CONFIRMED   |RESOLVED

--- Comment #20 from Nate Graham  ---
I'm going to mark this bug as a duplicate of Bug 315552 so we have everything
tracked in the same place. I encourage everyone who cast votes for this to move
their votes to 315552, because Bugzilla doesn't do it automatically.

Let's try to avoid extended arguments going forward. We all want this feature,
but the thing that will make it happen fastest is a patch coupled with
respectful dialogue.

*** This bug has been marked as a duplicate of bug 315552 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 315552] Allow save annotations directly to original pdf file

[Christoph Feck]

https://bugs.kde.org/show_bug.cgi?id=315552

--- Comment #20 from Christoph Feck  ---
Nate, maybe bug 301774?

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 385943] Feature Request: Option to save annotations with regular save

[Christoph Feck]

https://bugs.kde.org/show_bug.cgi?id=385943

Christoph Feck  changed:

   What|Removed |Added

 Status|UNCONFIRMED |RESOLVED
 Resolution|--- |DUPLICATE

--- Comment #1 from Christoph Feck  ---


*** This bug has been marked as a duplicate of bug 315552 ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 315552] Allow save annotations directly to original pdf file

[Christoph Feck]

https://bugs.kde.org/show_bug.cgi?id=315552

Christoph Feck  changed:

   What|Removed |Added

 CC||j.gjor...@gmail.com

--- Comment #19 from Christoph Feck  ---
*** Bug 385943 has been marked as a duplicate of this bug. ***

-- 
You are receiving this mail because:
You are the assignee for the bug.

[okular] [Bug 385943] New: Feature Request: Option to save annotations with regular save

[bugzilla_noreply]

https://bugs.kde.org/show_bug.cgi?id=385943

Bug ID: 385943
   Summary: Feature Request: Option to save annotations with
regular save
   Product: okular
   Version: unspecified
  Platform: Other
OS: Linux
Status: UNCONFIRMED
  Severity: normal
  Priority: NOR
 Component: general
  Assignee: okular-devel@kde.org
  Reporter: j.gjor...@gmail.com
  Target Milestone: ---

Currently if i want to save the annotations in the same file so i can use them
on other computers i have to go and Save As the documents, it would be great if
there is a checkbox that turns the default Save action to also save
annotations.

-- 
You are receiving this mail because:
You are the assignee for the bug.