[Bug 513541] Review Request: cpulimit - CPU Usage Limiter for Linux

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

 



Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug.


https://bugzilla.redhat.com/show_bug.cgi?id=513541





--- Comment #19 from Jussi Lehtola <jussi.lehtola@xxxxxx>  2009-07-29 08:26:24 EDT ---
(In reply to comment #18)
> (In reply to comment #16)
> > Don't bother with autotools. There's nothing wrong with a package that builds
> > without them, especially when the build process is this simple.
> 
> I agree, but my recommendation is a small patch to makefile without auto-tools.
> and sending this small patch to upstream of cpulimit.  

Yes, that is fine too. Then the makefile should look like


 BINDIR=/usr/local/bin
 MANDIR=/usr/local/man
 INSTALL="install"
 CFLAGS="-Wall -O2"

 all::   cpulimit

 cpulimit:       cpulimit.c
         gcc ${CFLAGS} -o cpulimit cpulimit.c -lrt

 clean:
         rm -f cpulimit

 install:
         ${INSTALL} -D -m 755 cpulimit ${DESTDIR}${BINDIR}
         ${INSTALL} -D -m 644 cpulimit.1 ${DESTDIR}${MANDIR}/man1/cpulimit.1


By default this will compile with "-Wall -O2" and install to /usr/local/bin. In
the spec file one would use

 %build
 make %{?_smp_mflags} CFLAGS="$RPM_OPT_FLAGS"
to override the CFLAGS variable in Makefile and

 %install
 make DESTDIR=$RPM_BUILD_ROOT BINDIR=%{_bindir} MANDIR=%{_mandir}
INSTALL="install -p" install
to override the other variables. One can also do without DESTDIR support, then
one just prepends $RPM_BUILD_ROOT to MANDIR and BINDIR.

-- 
Configure bugmail: https://bugzilla.redhat.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.

_______________________________________________
Fedora-package-review mailing list
Fedora-package-review@xxxxxxxxxx
http://www.redhat.com/mailman/listinfo/fedora-package-review

[Index of Archives]     [Fedora Legacy]     [Fedora Desktop]     [Fedora SELinux]     [Yosemite News]     [KDE Users]     [Fedora Tools]