[Debian-med-packaging] Bug#701801: New Upstream Snapshot and use of CMake Buildsystem

[Mathieu Malaterre]

Reinhard,

On Wed, Feb 27, 2013 at 3:50 PM, Andreas Tille <andreas at fam-tille.de> wrote:
> Hi Reinhard,
>
> On Wed, Feb 27, 2013 at 12:32:13PM +0100, Reinhard Tartler wrote:
>> It turns out that most patches are necessary for fixing the
>> automake-based build system. AFAIUI, upstream now prefers the use of
>> the CMake Build system, and in fact, most patches are no longer
>> necessary when the packaging uses that. Therefore, I have changed the
>> packaging to dh7 and cmake. This allowed me to simplify debian/rules
>> drastically.
>
> All these changes sound pretty reasonable and provided that Mathieu
> confirms his agreement I'd say it makes perfectly sense if you simply
> commit your changes into SVN.  Any DD has commit permissions to our
> SVN so this should not be a permission problem.


Make sure to use this URL when commiting:

svn+ssh://svn.debian.org/svn/debian-med/trunk/packages/dcmtk/branches/experimental

Until debian actually goes out of freeze, I do not want anything in
unstable. Too much work when something gets wrong.

>> Also, I had to rework the
>> debian/{dcmtk-www,libdcmtk2,libdcmtk2-dev}.install dh_install
>> configuration files. I could drop almost all patches. The only patches
>> that IMO still make sense are "01_fix_perl_script_path.patch",
>> "02_dcmtk_3.6.0-1.patch" and "prefs.patch".
>
> Sounds great.

Sound reasonable. 07_doxygen.patch should still be present however.

>> The only caveat that I did not manage to implement was the correct
>> construction of the dcmtk-doc package, because I was unable to locate
>> the relevant calls for invoking doxygen. Therefore, I have removed
>> dcmtk-doc from debian/control (it is not relevant to the needs of my
>> customer anyways).
>
> Mathieu would you consider checking this out?  I personally also do not
> have any cmake / doxygen experience but it would probably not to hard to
> do (as last resort with help of debian-mentors).

Please leave the doc package as-is, even if empty. I'll fix it ASAP.

>> Please find the relevant packaging files attached to this email.
>
> Thanks for you patches and as said above please consider direct commit.

Please commit to exp branch.

Thanks,
-- 
Mathieu