Re: [Rpm-maint] [rpm-software-management/rpm] Override ghost+dir filesizes (#229)

2017-06-06 Thread Bernhard M. Wiedemann
Closed #229.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/229#event-1112101274___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


Re: [Rpm-maint] [rpm-software-management/rpm] Override ghost+dir filesizes (#229)

2017-06-06 Thread Panu Matilainen
NAK, there are people actually relying on %ghost sizes being recorded and taken 
into account on free disk space checking. That said, in MOST cases the %ghost 
size is simply fubar and we'd be better off not recording it. I could imagine 
something like %ghost(size=1234K) but even that is breaking backwards 
compatibility.

For directories, simply setting size to zero in packages would probably be the 
least wrong value - the size of directory is an implementation detail of the 
installation-time target filesystem and build-time doesn't have a clue. And 
many of the directories that rpm deals with already exist on the fs, but again 
the build-time doesn't have a clue.

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/229#issuecomment-306451864___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint


[Rpm-maint] [rpm-software-management/rpm] Override ghost+dir filesizes (#229)

2017-06-02 Thread Bernhard M. Wiedemann
for directories and ghost files we do not want to
record their size in binary rpms
to make builds more reproducible.

See https://reproducible-builds.org/ for why this matters.

Note: I'm not sure if this is the best/correct way to do this, but at least it 
made the packages build bit-identical where they did not before.

To test add to %install section:
```
cd %{buildroot}/some/packaged/dir
for i in $(seq 1 $RANDOM) ; do touch xxx$i ; done
rm xxx*
```
You can view, comment on, or merge this pull request online at:

  https://github.com/rpm-software-management/rpm/pull/229

-- Commit Summary --

  * Override ghost+dir filesizes

-- File Changes --

M build/files.c (3)

-- Patch Links --

https://github.com/rpm-software-management/rpm/pull/229.patch
https://github.com/rpm-software-management/rpm/pull/229.diff

-- 
You are receiving this because you are subscribed to this thread.
Reply to this email directly or view it on GitHub:
https://github.com/rpm-software-management/rpm/pull/229
___
Rpm-maint mailing list
Rpm-maint@lists.rpm.org
http://lists.rpm.org/mailman/listinfo/rpm-maint