Re: SPECS: qt4.spec - adapter
Dnia środa, 20 czerwca 2007, glen napisał:
Author: glen Date: Wed Jun 20 21:51:01 2007 GMT
Module: SPECS Tag: HEAD
Log message:
- adapter
Files affected:
SPECS:
qt4.spec (1.134 - 1.135)
Diffs:
Index: SPECS/qt4.spec
diff -u SPECS/qt4.spec:1.134 SPECS/qt4.spec:1.135
--- SPECS/qt4.spec:1.134 Fri Jun 15 07:42:03 2007
+++ SPECS/qt4.specWed Jun 20 23:50:56 2007
@@ -1537,19 +1537,33 @@
%{_qtdir}/doc
%files -n QtCore-devel -f QtCore-devel.files
+%defattr(644,root,root,755)
^^
This change is wrong. Qt*-devel.files already contains this and it's
respected. If this is done by adapter, then adapter needs to be fixed.
Remove these lines, please.
Regards,
--
Rafał Cygnarowski
[EMAIL PROTECTED]
signature.asc
Description: This is a digitally signed message part.
___
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: SPECS: qt4.spec - adapter
Rafał Cygnarowski [EMAIL PROTECTED] [21-06-2007 10:14]: Dnia środa, 20 czerwca 2007, glen napisał: [...] %files -n QtCore-devel -f QtCore-devel.files +%defattr(644,root,root,755) ^^ This change is wrong. Why do you think so? Qt*-devel.files already contains this and it's respected. If this is done by adapter, then adapter needs to be fixed. Remove these lines, please. All %files sections should begin with %defattr; no exceptions. This cuts down the time needed to wonder if it's needed here and just missing or defined somewhere else... Consider it a policy. Benefit: less bugs, saved time. -- Radosław Zieliński [EMAIL PROTECTED] pgpm2pEKhbPZN.pgp Description: PGP signature ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: SPECS: qt4.spec - adapter
On Thursday 21 June 2007, Radoslaw Zielinski wrote: Rafał Cygnarowski [EMAIL PROTECTED] [21-06-2007 10:14]: Dnia środa, 20 czerwca 2007, glen napisał: [...] %files -n QtCore-devel -f QtCore-devel.files +%defattr(644,root,root,755) ^^ This change is wrong. Why do you think so? Qt*-devel.files already contains this and it's respected. If this is done by adapter, then adapter needs to be fixed. Remove these lines, please. All %files sections should begin with %defattr; no exceptions. This cuts down the time needed to wonder if it's needed here and just missing or defined somewhere else... Consider it a policy. Benefit: less bugs, saved time. agree (that's why i commited it too), and it made branch diffing easier of both branches in comparision are adaptered. the same topic can be argued around any *-i18n* *-i10l* packages. and extra note: it doesn't break anything, the %defattr from -f overrides it anyway. -- glen ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: SPECS: qt4.spec - adapter
Dnia czwartek, 21 czerwca 2007, Radoslaw Zielinski napisał: Rafał Cygnarowski [EMAIL PROTECTED] [21-06-2007 10:14]: Dnia środa, 20 czerwca 2007, glen napisał: [...] %files -n QtCore-devel -f QtCore-devel.files +%defattr(644,root,root,755) ^^ This change is wrong. Why do you think so? Because 1. it's already there 2. it's less readable now (so bugs prone). Qt*-devel.files already contains this and it's respected. If this is done by adapter, then adapter needs to be fixed. Remove these lines, please. All %files sections should begin with %defattr; no exceptions. And there is no exception in this situation. This cuts down the time needed to wonder if it's needed here and just missing or defined somewhere else... Consider it a policy. If someone(tm) took care of it to be right, then it's not policy but too smart adapter... In the same way I could check every thing 100 times to be sure that one variable is set correctly but one place is enough if done right in right place. Regards, -- Rafał Cygnarowski [EMAIL PROTECTED] signature.asc Description: This is a digitally signed message part. ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: SPECS: qt4.spec - adapter
Dnia czwartek, 21 czerwca 2007, Elan Ruusamäe napisał: agree (that's why i commited it too), and it made branch diffing easier of both branches in comparision are adaptered. On AC-branch it's also included in *-files so it's not an argument. the same topic can be argued around any *-i18n* *-i10l* packages. And it is... if there is this definition already then why makes it double? Besides it makes spec-s more readable and a little bit smaller. I understand including this in %files when file list is only addition, but when it's only one source of file list - it's not necessary. Regards, -- Rafał Cygnarowski [EMAIL PROTECTED] signature.asc Description: This is a digitally signed message part. ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
broken pl LC_TIME
What the f* is that? [EMAIL PROTECTED] /etc]# LC_TIME=pl_PL date Cz, 21 VI 2007, 18:02:04 CEST XICIIVIM months are archaic in polish. -- Tom Pala [EMAIL PROTECTED] http://vfmg.sourceforge.net/ http://tccs.sourceforge.net/ ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: broken pl LC_TIME
On Thu, Jun 21, 2007 at 06:02:44PM +0200, Tomasz Pala wrote: What the f* is that? [EMAIL PROTECTED] /etc]# LC_TIME=pl_PL date Cz, 21 VI 2007, 18:02:04 CEST XICIIVIM months are archaic in polish. http://lists.pld-linux.org/mailman/pipermail/pld-devel-pl/2007-June/141423.html -- Jakub Boguszhttp://qboosh.pl/ ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: SPECS: qt4.spec - adapter
=?utf-8?q?Rafa=C5=82_Cygnarowski?= wrote: --===0990176345== Content-Type: multipart/signed; boundary=nextPart5329749.Eslldous2x; protocol=application/pgp-signature; micalg=pgp-sha1 Content-Transfer-Encoding: 7bit %files -n QtCore-devel -f QtCore-devel.files +%defattr(644,root,root,755) ^^ This change is wrong. Qt*-devel.files already contains this and it's=20 respected. If this is done by adapter, then adapter needs to be fixed. Remove these lines, please. It was told some times ago that the rule is to have the %defattr line just after %files. Even if file used via -f (generally language file) already contains it - just for case the -f option is temporary disabled for any reason. This may be a little overloading but should not hurt. -- === Andrzej M. Krzysztofowicz [EMAIL PROTECTED] phone (48)(58) 347 19 36 Faculty of Applied Phys. Math., Gdansk University of Technology ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: broken pl LC_TIME
Dnia 21-06-2007, Cz o godzinie 19:03 +0200, Jakub Bogusz napisał(a): On Thu, Jun 21, 2007 at 06:02:44PM +0200, Tomasz Pala wrote: What the f* is that? [EMAIL PROTECTED] /etc]# LC_TIME=pl_PL date Cz, 21 VI 2007, 18:02:04 CEST XICIIVIM months are archaic in polish. http://lists.pld-linux.org/mailman/pipermail/pld-devel-pl/2007-June/141423.html ok, but $ cal czerwiec 2007 N Pn Wt Śr Cz Pt So 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 one letter for Sunday is ... TiweK -- ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: broken pl LC_TIME
On Thursday 21 June 2007 22:06, Tomasz Witek wrote: ok, but $ cal czerwiec 2007 N Pn Wt Śr Cz Pt So 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 one letter for Sunday is ... what? we have all days single letter ;) $ LC_ALL= LC_TIME=et_EE cal -m juuni 2007 E T K N R L P 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 TiweK -- glen ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: broken pl LC_TIME
On 6/21/07, Tomasz Witek [EMAIL PROTECTED] wrote: ok, but $ cal czerwiec 2007 N Pn Wt Śr Cz Pt So 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 one letter for Sunday is ... ...perfectly normal? Or do you prefer the knights who call Sunday Ni? -- Patryk Zawadzki Generated Content ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: SPECS: qt4.spec - adapter
Dnia czwartek, 21 czerwca 2007, Andrzej Krzysztofowicz napisał: It was told some times ago that the rule is to have the %defattr line just after %files. Even if file used via -f (generally language file) already contains it - just for case the -f option is temporary disabled for any reason. This may be a little overloading but should not hurt. Adapter starts remind me MS Word autocorrection (not every dot followed by space is new sentence!). -- Rafał Cygnarowski [EMAIL PROTECTED] signature.asc Description: This is a digitally signed message part. ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
createrepo slowness - reconsidering yum indexes in pld th support
Hello,
Can createrepo index update process can be made faster? It takes about *1
hour* to update Th indexes (where poldek does the same thing in less that 1
minute for pndir format).
Currently it's called as:
os.system('%s.stat/bin/createrepo --cache %s/tmp/createrepo %s%s/%s/RPMS' %
(ftp_dir,home,ftp_dir,tree,arch))
using ep09 packages (pld ac).
--
Arkadiusz MiśkiewiczPLD/Linux Team
arekm / maven.plhttp://ftp.pld-linux.org/
___
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: broken pl LC_TIME
On Thu, Jun 21, 2007 at 10:26:00PM +0300, Elan Ruusamäe wrote:
On Thursday 21 June 2007 22:06, Tomasz Witek wrote:
ok, but
$ cal
czerwiec 2007
N Pn Wt Śr Cz Pt So
one letter for Sunday is ...
what? we have all days single letter ;)
$ LC_ALL= LC_TIME=et_EE cal -m
juuni 2007
E T K N R L P
and here all week days start by letter 'd', so this:
juny de 2007
dl dt dc dj dv ds dg
1 2 3
should be considered 2 or one letter ? :P
--
Sparky{PI] -- Przemyslaw _ ___ _ _ ... LANG...Pl..Ca..Es..En
/) ___ ___ _ _ || Iskra | | _ \| | | : WWWppcrcd.pld-linux.org
\\| -_)'___| ||^'||//\\//| _/| | | : JID..sparkyatjabberes.org
(/|| (_-_|_|| ||\\ || |_ |_| |_| _| : Mailsparkyatpld-linux.org
___
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: NEW poldek 0.20070618.11.1 (Ac)
On 6/20/07, Elan Ruusamäe [EMAIL PROTECTED] wrote: On Wednesday 20 June 2007, kiesiu wrote: Any suggestions? https://bugs.pld-linux.org/show_bug.cgi?id=25 Another goodie: Zapisywanie /home/users/patrys/tmp/[...]/packages.ndir.gz... Błąd w obliczeniach zmiennoprzecinkowych Cannot reproduce (above says floating point arithmetic error) -- Patryk Zawadzki Generated Content ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
Re: broken pl LC_TIME
Tomasz Witek wrote: Dnia 21-06-2007, Cz o godzinie 21:33 +0200, Patryk Zawadzki napisaĹ(a): On 6/21/07, Tomasz Witek [EMAIL PROTECTED] wrote: ok, but $ cal czerwiec 2007 N Pn Wt Ĺr Cz Pt So 1 2 3 4 5 6 7 8 9 10 11 12 13 14 15 16 17 18 19 20 21 22 23 24 25 26 27 28 29 30 one letter for Sunday is ... ...perfectly normal? Or do you prefer the knights who call Sunday Ni? Why if all have two ? Nd ? N or N should solve it. -- === Andrzej M. Krzysztofowicz [EMAIL PROTECTED] phone (48)(58) 347 19 36 Faculty of Applied Phys. Math., Gdansk University of Technology ___ pld-devel-en mailing list pld-devel-en@lists.pld-linux.org http://lists.pld-linux.org/mailman/listinfo/pld-devel-en
