Re: [packages/tzdata] break dependency loop

2017-04-04 Thread Elan Ruusamäe 



On 04/04/17 15:55, Elan Ruusamäe wrote:


On 04/04/17 11:05, Jacek Konieczny wrote:


Then tzdata-zoneinfo should not require the main package through the 
/etc/localtime symlink. 


so, your change was to move /etc/locatime symlink and related files to 
main package.


the only thing more needed is to own dir in one package only.


i had this change in mind

https://github.com/pld-linux/tzdata/commit/f8a6a5a8a6b816673dfe9551ddf5c4afb7cd2dbe


committed to fixup-2141fdb branch
___
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en

Re: [packages/tzdata] break dependency loop

2017-04-04 Thread Elan Ruusamäe 


On 04/04/17 11:05, Jacek Konieczny wrote:


Then tzdata-zoneinfo should not require the main package through the 
/etc/localtime symlink. 


so, your change was to move /etc/locatime symlink and related files to 
main package.


the only thing more needed is to own dir in one package only.

___
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en

Re: [packages/tzdata] break dependency loop

2017-04-04 Thread Jacek Konieczny 

On 2017-04-03 21:23, Elan Ruusamäe wrote:

On 13.03.2017 10:30, jajcus wrote:

commit 2141fdbf9a858852a7c635341b346a5ca42787f1
Author: Jacek Konieczny 
Date:   Mon Mar 13 09:26:54 2017 +0100

 break dependency loop
  warning: LOOP:
 warning: removing tzdata-zoneinfo-2016j-1.aos1.noarch "Requires:
/etc/localtime" from tsort relations.
 warning: removing tzdata-2016j-1.aos1.noarch "Requires:
tzdata-zoneinfo = 2016j-1.aos1" from tsort relations.
  This breaks package install order on clean systems.
  Release: 2


[...]


now %{_datadir}/zoneinfo is packaged on both packages (tzdata and
tzdata-zoneinfo)

main package should not include it as main package rewquires
tzdata-zoneinfo anyway.


Then tzdata-zoneinfo should not require the main package through  the 
/etc/localtime symlink.



I think this is all to complicated anyway:
- /etc/rc.d/init.d/timezone should be a part of rc-scripts, only as a 
fallback for systemd-less systems (systemd does handle timezones better 
and has timedatectl)
- there is no need for the tzdata/tzdata-zoneinfo  split. Now we have 
'tzdata' package, which contains no 'data' and is quite obsolete on 
modern systems
- /etc/localtime should not be a symlink. systemd does not make it a 
symlink and it being a symlink causes potential problems (/usr 
unmounted, change in zone file names).



One 'tzdata' package only with the data and no external dependencies 
would be enough.


Jacek
___
pld-devel-en mailing list
pld-devel-en@lists.pld-linux.org
http://lists.pld-linux.org/mailman/listinfo/pld-devel-en