Date: Mon, 21 Jan 2019 11:41:52 +0100 From: =?UTF-8?B?VMSzbA==?= Coosemans <tijl@FreeBSD.org> To: "Alexandre C. =?UTF-8?B?R3VpbWFyw6Nlcw==?=" <rigoletto@FreeBSD.org> Cc: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: Re: svn commit: r490800 - in head/net-p2p: transmission-cli transmission-cli/files transmission-daemon transmission-gtk transmission-qt Message-ID: <20190121114152.053d827c@kalimero.tijl.coosemans.org> In-Reply-To: <201901201702.x0KH2RQu011803@repo.freebsd.org> References: <201901201702.x0KH2RQu011803@repo.freebsd.org>
next in thread | previous in thread | raw e-mail | index | archive | help
On Sun, 20 Jan 2019 17:02:27 +0000 (UTC) Alexandre C. Guimar=C3=A3es <rigol= etto@FreeBSD.org> wrote: > Author: rigoletto > Date: Sun Jan 20 17:02:27 2019 > New Revision: 490800 > URL: https://svnweb.freebsd.org/changeset/ports/490800 >=20 > Log: > net-p2p/transmission-cli: change transmission's data size unit > conversion factors from 1000 to 1024, to match FreeBSD's blocksize. What blocksize? The disk block sizes are determined by the hardware so FreeBSD isn't any different from Linux. Why should the application behave differently on FreeBSD than it does on Linux? > Added: head/net-p2p/transmission-cli/files/patch-Units > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- /dev/null 00:00:00 1970 (empty, because file is newly added) > +++ head/net-p2p/transmission-cli/files/patch-Units Sun Jan 20 17:02:27 2= 019 (r490800) > @@ -0,0 +1,152 @@ > +--- cli/cli.c.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ cli/cli.c 2019-01-18 17:21:27.893302000 -0800 > +@@ -46,14 +46,14 @@ > + #define MEM_G_STR "GiB" > + #define MEM_T_STR "TiB" Since they use GiB and TiB here... > +-#define DISK_K 1000 > ++#define DISK_K 1024 > + #define DISK_B_STR "B" > + #define DISK_K_STR "kB" > + #define DISK_M_STR "MB" > + #define DISK_G_STR "GB" > + #define DISK_T_STR "TB" ...you should use KiB, MiB, GiB and TiB here... > +-#define SPEED_K 1000 > ++#define SPEED_K 1024 > + #define SPEED_B_STR "B/s" > + #define SPEED_K_STR "kB/s" > + #define SPEED_M_STR "MB/s" ...and here as well (although using 1024 for bandwidth is weird). Similar for all cases below. But really I think you should revert this change. If the submitter wants this he can discuss that with the transmission developers. > +--- daemon/daemon.c.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ daemon/daemon.c 2019-01-18 17:21:55.699919000 -0800 > +@@ -50,14 +50,14 @@ > + #define MEM_G_STR "GiB" > + #define MEM_T_STR "TiB" > +=20 > +-#define DISK_K 1000 > ++#define DISK_K 1024 > + #define DISK_B_STR "B" > + #define DISK_K_STR "kB" > + #define DISK_M_STR "MB" > + #define DISK_G_STR "GB" > + #define DISK_T_STR "TB" > +=20 > +-#define SPEED_K 1000 > ++#define SPEED_K 1024 > + #define SPEED_B_STR "B/s" > + #define SPEED_K_STR "kB/s" > + #define SPEED_M_STR "MB/s" > +--- daemon/remote.c.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ daemon/remote.c 2019-01-18 18:03:10.333078000 -0800 > +@@ -44,14 +44,14 @@ > + #define MEM_G_STR "GiB" > + #define MEM_T_STR "TiB" > +=20 > +-#define DISK_K 1000 > ++#define DISK_K 1024 > + #define DISK_B_STR "B" > + #define DISK_K_STR "kB" > + #define DISK_M_STR "MB" > + #define DISK_G_STR "GB" > + #define DISK_T_STR "TB" > +=20 > +-#define SPEED_K 1000 > ++#define SPEED_K 1024 > + #define SPEED_B_STR "B/s" > + #define SPEED_K_STR "kB/s" > + #define SPEED_M_STR "MB/s" > +--- gtk/util.c.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ gtk/util.c 2019-01-18 17:23:10.918334000 -0800 > +@@ -39,13 +39,13 @@ > + const char * mem_G_str =3D N_("GiB"); > + const char * mem_T_str =3D N_("TiB"); > +=20 > +-const int disk_K =3D 1000; > ++const int disk_K =3D 1024; > + const char * disk_K_str =3D N_("kB"); > + const char * disk_M_str =3D N_("MB"); > + const char * disk_G_str =3D N_("GB"); > + const char * disk_T_str =3D N_("TB"); > +=20 > +-const int speed_K =3D 1000; > ++const int speed_K =3D 1024; > + const char * speed_K_str =3D N_("kB/s"); > + const char * speed_M_str =3D N_("MB/s"); > + const char * speed_G_str =3D N_("GB/s"); > +--- libtransmission/libtransmission-test.c.orig 2018-05-01 10:18:29.0000= 00000 -0700 > ++++ libtransmission/libtransmission-test.c 2019-01-18 17:23:43.535466000= -0800 > +@@ -200,14 +200,14 @@ > + #define MEM_G_STR "GiB" > + #define MEM_T_STR "TiB" > +=20 > +-#define DISK_K 1000 > ++#define DISK_K 1024 > + #define DISK_B_STR "B" > + #define DISK_K_STR "kB" > + #define DISK_M_STR "MB" > + #define DISK_G_STR "GB" > + #define DISK_T_STR "TB" > +=20 > +-#define SPEED_K 1000 > ++#define SPEED_K 1024 > + #define SPEED_B_STR "B/s" > + #define SPEED_K_STR "kB/s" > + #define SPEED_M_STR "MB/s" > +--- qt/Formatter.cc.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ qt/Formatter.cc 2019-01-18 17:25:58.166361000 -0800 > +@@ -29,7 +29,7 @@ > + void > + Formatter::initUnits () > + { > +- speed_K =3D 1000; > ++ speed_K =3D 1024; > + unitStrings[SPEED][B] =3D tr ( "B/s"); > + unitStrings[SPEED][KB] =3D tr ("kB/s"); > + unitStrings[SPEED][MB] =3D tr ("MB/s"); > +@@ -41,7 +41,7 @@ > + unitStrings[SPEED][GB].toUtf8().constData(), > + unitStrings[SPEED][TB].toUtf8().constData()); > +=20 > +- size_K =3D 1000; > ++ size_K =3D 1024; > + unitStrings[SIZE][B] =3D tr ( "B"); > + unitStrings[SIZE][KB] =3D tr ("kB"); > + unitStrings[SIZE][MB] =3D tr ("MB"); > +--- utils/units.h.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ utils/units.h 2019-01-18 17:26:23.460607000 -0800 > +@@ -16,14 +16,14 @@ > + #define MEM_G_STR "GiB" > + #define MEM_T_STR "TiB" > +=20 > +-#define DISK_K 1000 > ++#define DISK_K 1024 > + #define DISK_B_STR "B" > + #define DISK_K_STR "kB" > + #define DISK_M_STR "MB" > + #define DISK_G_STR "GB" > + #define DISK_T_STR "TB" > +=20 > +-#define SPEED_K 1000 > ++#define SPEED_K 1024 > + #define SPEED_B_STR "B/s" > + #define SPEED_K_STR "kB/s" > + #define SPEED_M_STR "MB/s" > +--- web/javascript/formatter.js.orig 2018-05-01 10:18:29.000000000 -0700 > ++++ web/javascript/formatter.js 2019-01-18 17:26:43.307090000 -0800 > +@@ -7,14 +7,14 @@ > +=20 > + Transmission.fmt =3D (function() > + { > +- var speed_K =3D 1000; > ++ var speed_K =3D 1024; > + var speed_B_str =3D 'B/s'; > + var speed_K_str =3D 'kB/s'; > + var speed_M_str =3D 'MB/s'; > + var speed_G_str =3D 'GB/s'; > + var speed_T_str =3D 'TB/s'; > +=20 > +- var size_K =3D 1000; > ++ var size_K =3D 1024; > + var size_B_str =3D 'B'; > + var size_K_str =3D 'kB'; > + var size_M_str =3D 'MB'; >=20 > Modified: head/net-p2p/transmission-daemon/Makefile > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/net-p2p/transmission-daemon/Makefile Sun Jan 20 14:56:18 2019 (r= 490799) > +++ head/net-p2p/transmission-daemon/Makefile Sun Jan 20 17:02:27 2019 (r= 490800) > @@ -1,6 +1,6 @@ > # $FreeBSD$ > =20 > -PORTREVISION=3D 1 > +PORTREVISION=3D 2 > PKGNAMESUFFIX=3D -daemon > =20 > USES=3D ssl >=20 > Modified: head/net-p2p/transmission-gtk/Makefile > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/net-p2p/transmission-gtk/Makefile Sun Jan 20 14:56:18 2019 (r490= 799) > +++ head/net-p2p/transmission-gtk/Makefile Sun Jan 20 17:02:27 2019 (r490= 800) > @@ -1,7 +1,6 @@ > # $FreeBSD$ > =20 > -PORTREVISION=3D 2 > - > +PORTREVISION=3D 3 > PKGNAMESUFFIX=3D -gtk > =20 > LIB_DEPENDS=3D libdbus-glib-1.so:devel/dbus-glib \ >=20 > Modified: head/net-p2p/transmission-qt/Makefile > =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D= =3D=3D=3D=3D > --- head/net-p2p/transmission-qt/Makefile Sun Jan 20 14:56:18 2019 (r4907= 99) > +++ head/net-p2p/transmission-qt/Makefile Sun Jan 20 17:02:27 2019 (r4908= 00) > @@ -1,6 +1,6 @@ > # $FreeBSD$ > =20 > -PORTREVISION=3D 5 > +PORTREVISION=3D 6 > PKGNAMESUFFIX=3D -${FLAVOR} > =20 > LIB_DEPENDS=3D libfontconfig.so:x11-fonts/fontconfig \ >=20
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?20190121114152.053d827c>