MantisBT - CMake
View Issue Details
IDProjectCategoryView StatusDate SubmittedLast Update
0010296CMakeCPackpublic2010-02-16 09:202010-09-09 23:58
ReporterDroscy 
Assigned ToEric NOULARD 
PrioritynormalSeverityminorReproducibilityalways
StatusclosedResolutionfixed 
PlatformOSOS Version
Product VersionCMake-2-8 
Target VersionCMake 2.8.3Fixed in VersionCMake 2.8.3 
Summary0010296: Wrong "Installed-Size" field in generated Debian packages
DescriptionI noticed that the field "Installed-Size" in the control file of the generated Debian packages reports the size in bytes, while Debian considers that size in kilobytes, so a program of 200KiB appears to be 200MiB.

Attached here the patch for the file Source/CPack/cmCPackDebGenerator.cxx
Steps To Reproduce
Additional Information
TagsNo tags attached.
Relationships
related to 0010292closed Eric NOULARD Patch for CPack Debian generator (auto-dependecy support via dpkg-shlibdeps) 
has duplicate 0008862closed Eric NOULARD Wrong Installed-Size field in Debian-Package 
Attached Filespatch installed-size.patch (497) 2010-02-16 09:20
https://public.kitware.com/Bug/file/2870/installed-size.patch
Issue History
Date ModifiedUsernameFieldChange
2010-02-16 09:20DroscyNew Issue
2010-02-16 09:20DroscyFile Added: installed-size.patch
2010-05-17 17:35Modestas VainiusNote Added: 0020768
2010-05-18 04:54DroscyNote Added: 0020778
2010-05-18 06:38Eric NOULARDRelationship addedrelated to 0010292
2010-07-28 11:05Eric NOULARDStatusnew => assigned
2010-07-28 11:05Eric NOULARDAssigned To => Eric NOULARD
2010-07-28 11:06Eric NOULARDNote Added: 0021513
2010-07-28 11:06Eric NOULARDNote Added: 0021514
2010-07-28 11:06Eric NOULARDStatusassigned => closed
2010-07-28 11:06Eric NOULARDResolutionopen => fixed
2010-07-28 11:06Eric NOULARDFixed in Version => CMake-2-8
2010-07-29 11:53Eric NOULARDRelationship addedhas duplicate 0008862
2010-09-09 23:58David ColeFixed in VersionCMake-2-8 => CMake 2.8.3
2010-09-09 23:58David ColeTarget Version => CMake 2.8.3

Notes
(0020768)
Modestas Vainius   
2010-05-17 17:35   
I have included a bit better version of the patch in the cmake debian package. Debian Policy requires the value to be "rounded up", e.g. 1000 bytes should result in 1, not in 0 as in the original patch. See

http://git.debian.org/?p=collab-maint/cmake.git;a=blob;f=debian/patches/cpack_installed_size.diff [^]
(0020778)
Droscy   
2010-05-18 04:54   
I was not aware of that policy. Thanks for you improvments :-)
(0021513)
Eric NOULARD   
2010-07-28 11:06   
Thank you for your contribution.
Patch tested and merged to next:
commit c1cf11275df47a8eff96e13322b5df065c027b39
Merge: 5514d80 5495a6d
Author: Eric NOULARD <eric.noulard@gmail.com>
Date: Wed Jul 28 17:04:45 2010 +0200

    Merge branch 'CPackDEB-wrong-install-size' into next

commit 5495a6d9756099d5eddfc99c938ab6ba9bd95113
Author: Eric NOULARD <eric.noulard@gmail.com>
Date: Wed Jul 28 17:03:14 2010 +0200

    CPackDEB: merge wrong installed size patch. see bugs 10296 (and 10292)
(0021514)
Eric NOULARD   
2010-07-28 11:06   
Just merged to next.