[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Bug#959702: marked as done (RFS: cpufrequtils/008-2 [ITA] -- utilities to deal with the cpufreq Linux kernel feature)

Your message dated Tue, 05 May 2020 09:12:13 -0400
with message-id <43ffa07c75999fdbb6b28b1eadb0fb9400def863.camel@debian.org>
and subject line Re: Bug#959702: RFS: cpufrequtils/008-2 [ITA] -- utilities to deal with the cpufreq Linux kernel feature
has caused the Debian Bug report #959702,
regarding RFS: cpufrequtils/008-2 [ITA] -- utilities to deal with the cpufreq Linux kernel feature
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
Bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact owner@bugs.debian.org
immediately.)


-- 
959702: https://bugs.debian.org/cgi-bin/bugreport.cgi?bug=959702
Debian Bug Tracking System
Contact owner@bugs.debian.org with problems
--- Begin Message --- 
Package: sponsorship-requests
Severity: normal

Dear mentors,

I am looking for a sponsor for my package "cpufrequtils"

 * Package name    : cpufrequtils
   Version         : 008-2
   Upstream Author : Dominik Brodowski <linux@dominikbrodowski.de>
 * URL             : http://kernel.org/pub/linux/utils/kernel/cpufreq/cpufrequtils.html
 * License         : GPL-2
 * Vcs             : https://salsa.debian.org/kkamagui-guest/cpufrequtils
   Section         : admin

It builds those binary packages:

  cpufrequtils - utilities to deal with the cpufreq Linux kernel feature
  libcpufreq0 - shared library to deal with the cpufreq Linux kernel feature
  libcpufreq-dev - development files to deal with the cpufreq Linux kernel feature

To access further information about this package, please visit the following URL:

  https://mentors.debian.net/package/cpufrequtils

Alternatively, one can download the package with dget using this command:

  dget -x https://mentors.debian.net/debian/pool/main/c/cpufrequtils/cpufrequtils_008-2.dsc

Changes since the last upload:

   * New maintainer (Closes: #933159)
   * Add "for internal use" to debian/cpufrequtils.templates to fix
     lintian warning.
   * Bump Standards-Version to 4.5.0.
   * Remove -O2 of CFLAGS from debian/rules when noopt is defined.
   * Add hardening+=all to debian/rules and set-hardening.patch.
   * Add Vcs-* fields to debian/control.
   * Add Build-Depends-Package field to debian/libcpufreq0.symbols.
   * Change the format of debian/copyright to dep5.
   * Fix tyops in man files.
   * Remove debian/source.lintian-overrides to fix lintian warning.

Regards,

--
  Seunghun Han

--- End Message ---
--- Begin Message --- 
Hi Seunghun,

Uploaded.

One last minor issue: you may replace the long sequence of architecture
(Architecture: alpha amd64 arm64 armel armhf hppa i386 ia64 m68k mips mips64el
mipsel s390x sh4 sparc64 x32) with "Architecture: any" to allow the package to
be built on any architecture. This make the package automatically available
whenever a new architecture emerges. Of course, if the package can only
reasonably work under certain architectures, it's better to explicitly list
out those archs.

-- 
Best,
Boyuan Yang

在 2020-05-05星期二的 19:22 +0900，Seunghun Han写道：
> Hi Boyuan,
> 
> Thank you for your advice. I have fixed all issues you said. Please
> see the comments below.
> 
> On Tue, May 5, 2020 at 11:37 AM Boyuan Yang <byang@debian.org> wrote:
> > The members of salsa.debian.org/debian group are limited to DDs (Debian
> > Developers). However, it is possible to add you as the maintainer of
> > individual repos under Debian/ group. I have already added you as a
> > maintainer
> > of https://salsa.debian.org/debian/cpufrequtils .
> > 
> 
> I really appreciate it. After your confirmation and sponsor, I will
> move all contents to the repository. The debian/control was already
> changed to the repo.
> 
> > As part of mentoring, I reviewed your files in the git repo and found some
> > other issues:
> > 
> > * You accidentally committed files under debian/cpufrequtils into the
> > repository. Those files are built binary files instead of source codes and
> > should not be included.
> >   - This also indicates that you are using a non-chroot environment to
> > build
> > packages. Please consider using clean chroot each time doing a package
> > build,
> > like pbuilder, cowbuilder, git-builder, sbuild, etc.
> 
> Sorry for my mistake. It was my fault, and I didn't intend to add the
> binary files. I will use the pbuilder for avoiding this kind of
> mistake.
> 
> > * In architecture field of package cpufrequtils, it lists "hurd", "kbsd64"
> > and
> > "kbsd32", which does not exist in Debian. Please correct it. (I know it
> > could
> > be a bug in older versions but we should be fixing it once we know it.)
> > 
> > * You may now use https://salsa.debian.org/debian/cpufrequtils in Vcs-*
> > fields.
> 
> I fixed these issues with your advice.
> 
> > * I'm not sure if debian/lintian.overrides will work.
> > https://manpages.debian.org/unstable/lintian/lintian.1.en.html#FILES
> > mentioned
> > correct naming about lintian override files.
> > 
> > After solving those problems, I will sponsor this package.
> 
> According to my lintian report, debian/lintian.overrides seemed to
> work. The brief report is below.
> $> lintian -i -I --pedantic cpufrequtils_008-2_amd64.changes
> [... snip ...]
> N: 1 tag overridden (1 error)     <= Maybe this point shows it works
> 
> > Some bonus points that can be done laster in future:
> > 
> > * Consider converting debian/rules to be using dh(1) sequencer later.
> 
> Thank you. I will keep this for the next item.
> 
> Best regards,
> 
> Seunghun

Attachment: signature.asc
Description: This is a digitally signed message part


--- End Message ---
Reply to: