search

tcurdt / jdeb

This library provides an Ant task and a Maven plugin to create Debian packages from Java builds in a truly cross platform manner.
https://github.com/tcurdt/jdeb
Apache License 2.0
485 stars 264 forks source link

Update Control Builder,ChangesFileBuilder,pom.xml,DebMaker #640

Open Turbocyb0rg opened 4 months ago

Turbocyb0rg commented 4 months ago

adding properties for the source and target 1.8 to 17 resolved the warning

import java.nio.charset.StandardCharsets; in ControlBuilder

few of the TODO's and FIXNeeded parts

NullOutputStream.NULL_OUTPUT_STREAM is deprecated changed to NullOutputStream.INSTANCE

because NullOutputStream have a field named INSTANCE, resolved the warning. in ChangesFileBuilder and ControlBuilder

String content = IOUtils.toString(in); changed to String content = IOUtils.toString(in, StandardCharsets.UTF_8); in ControlBuilder

this.signDigest = signDigest; changed to this.signDigest = digest;

changes to LsMapper private int convertModeFromString( final String mode )

changes in isSubmodules(), whether the current Maven project is a submodule or not

check if the entry name ends with a symbolic link extension (.symlink) i.e., if (src.getName().endsWith(".symlink"))

tcurdt commented 3 months ago

Thanks for the contribution - but it seems like this has build issues.

It would also be much better to create smaller and more focused PRs.