Re: [CVS] RPM: rpm-5_4: rpm/ CHANGES rpm/macros/ macros.rpmbuild.in rpm/sc...

2011-12-20 Thread Per Øyvind Karlsen
Den 17:39 21. oktober 2011 skrev Jeff Johnson j...@rpm5.org følgende:
  RPM Package Manager, CVS Repository
  http://rpm5.org/cvs/
  

  Server: rpm5.org                         Name:   Jeff Johnson
  Root:   /v/rpm/cvs                       Email:  j...@rpm5.org
  Module: rpm                              Date:   21-Oct-2011 17:39:03
  Branch: rpm-5_4                          Handle: 2011102115390101

  Modified files:           (Branch: rpm-5_4)
    rpm                     CHANGES
    rpm/macros              macros.rpmbuild.in
    rpm/scripts             find-debuginfo.sh

  Log:
    - debuginfo: use current dir instead of $RPM_BUILD_DIR.
D'oh, a bit late discovered, but better than never..

$RPM_BUILD_DIR == %{_builddir}
while
$PWD == %{_builddir}/%{?buildsubdir}

So for most packages which have their own %buildsubdir, this will mess
up the paths for debug packages, ie. see the following example:

[peroyvind@t61 linux_logo]$ rpm -qpl
/home/peroyvind/RPM/linux_logo/RPMS/x86_64/linux_logo-debug-5.11-1-mdv2012.0.x86_64.rpm
/usr/lib/debug
/usr/lib/debug/.build-id
/usr/lib/debug/.build-id/02
/usr/lib/debug/.build-id/02/da5f6707c052c4293ea8e055de33d2704a3933
/usr/lib/debug/.build-id/02/da5f6707c052c4293ea8e055de33d2704a3933.debug
/usr/lib/debug/usr
/usr/lib/debug/usr/bin
/usr/lib/debug/usr/bin/linux_logo.debug
/usr/src/debug/defaults.h
/usr/src/debug/home
/usr/src/debug/home/peroyvind
/usr/src/debug/home/peroyvind/RPM
/usr/src/debug/home/peroyvind/RPM/linux_logo
/usr/src/debug/home/peroyvind/RPM/linux_logo/BUILD
/usr/src/debug/home/peroyvind/RPM/linux_logo/BUILD/linux_logo-5.11
/usr/src/debug/libsysinfo-0.2.2
/usr/src/debug/libsysinfo-0.2.2/Linux
/usr/src/debug/libsysinfo-0.2.2/Linux/cpuinfo_x86.c
/usr/src/debug/libsysinfo-0.2.2/Linux/sysinfo_linux.c
/usr/src/debug/libsysinfo-0.2.2/all
/usr/src/debug/libsysinfo-0.2.2/all/fix_mhz.c
/usr/src/debug/libsysinfo-0.2.2/all/parsing.c
/usr/src/debug/libsysinfo-0.2.2/all/sysinfo_common.c
/usr/src/debug/libsysinfo-0.2.2/all/uname.c
/usr/src/debug/libsysinfo-0.2.2/sysinfo.h
/usr/src/debug/linux_logo.c
/usr/src/debug/linux_logo.h
/usr/src/debug/load_logo.c
/usr/src/debug/load_logos.h
/usr/src/debug/logo_types.h

--
Regards,
Per Øyvind
__
RPM Package Managerhttp://rpm5.org
Developer Communication Listrpm-devel@rpm5.org


Re: [CVS] RPM: rpm-5_4: rpm/ CHANGES rpm/macros/ macros.rpmbuild.in rpm/sc...

2011-12-20 Thread Jeffrey Johnson

On Dec 20, 2011, at 9:55 AM, Per Øyvind Karlsen wrote:

 Den 17:39 21. oktober 2011 skrev Jeff Johnson j...@rpm5.org følgende:
  RPM Package Manager, CVS Repository
  http://rpm5.org/cvs/
  
 
  Server: rpm5.org Name:   Jeff Johnson
  Root:   /v/rpm/cvs   Email:  j...@rpm5.org
  Module: rpm  Date:   21-Oct-2011 17:39:03
  Branch: rpm-5_4  Handle: 2011102115390101
 
  Modified files:   (Branch: rpm-5_4)
rpm CHANGES
rpm/macros  macros.rpmbuild.in
rpm/scripts find-debuginfo.sh
 
  Log:
- debuginfo: use current dir instead of $RPM_BUILD_DIR.
 D'oh, a bit late discovered, but better than never..
 
 $RPM_BUILD_DIR == %{_builddir}
 while
 $PWD == %{_builddir}/%{?buildsubdir}
 
 So for most packages which have their own %buildsubdir, this will mess
 up the paths for debug packages, ie. see the following example:
 

I hope … have their own %buildsubdir … doesn't mean that some
package monkey is trying to set/change %builddubdor in a *.spec file.

Meanwhile, I'm not sure what you have discovered: literally nothing has
changed in this area for most of this century, the whole mechanism
if fabulously broken and mis-designed and mis-implemented imho.

No change to any of these conventions should be undertaken, nor do I
have any interest in fixes short of scrapping everything and reimplementing
more soundly, and that effort cannot be done without a ROADMAP
and goals and automated testing and more.

But Have it your own way! feel free to change whatever you wish,
just don't check any wild hacking changes in.

hth

73 de Jeff__
RPM Package Managerhttp://rpm5.org
Developer Communication Listrpm-devel@rpm5.org


Re: [CVS] RPM: rpm-5_4: rpm/ CHANGES rpm/macros/ macros.rpmbuild.in rpm/sc...

2011-12-20 Thread Per Øyvind Karlsen
Den 16:16 20. desember 2011 skrev Jeffrey Johnson n3...@me.com følgende:

 On Dec 20, 2011, at 9:55 AM, Per Øyvind Karlsen wrote:

 Den 17:39 21. oktober 2011 skrev Jeff Johnson j...@rpm5.org følgende:
  RPM Package Manager, CVS Repository
  http://rpm5.org/cvs/
  

  Server: rpm5.org                         Name:   Jeff Johnson
  Root:   /v/rpm/cvs                       Email:  j...@rpm5.org
  Module: rpm                              Date:   21-Oct-2011 17:39:03
  Branch: rpm-5_4                          Handle: 2011102115390101

  Modified files:           (Branch: rpm-5_4)
    rpm                     CHANGES
    rpm/macros              macros.rpmbuild.in
    rpm/scripts             find-debuginfo.sh

  Log:
    - debuginfo: use current dir instead of $RPM_BUILD_DIR.
 D'oh, a bit late discovered, but better than never..

 $RPM_BUILD_DIR == %{_builddir}
 while
 $PWD == %{_builddir}/%{?buildsubdir}

 So for most packages which have their own %buildsubdir, this will mess
 up the paths for debug packages, ie. see the following example:


 I hope … have their own %buildsubdir … doesn't mean that some
 package monkey is trying to set/change %builddubdor in a *.spec file.
Nope, I'm referring to whether %buildsubdir is set or not (as
controlled by %setup).
For most packages it's set, with the default being %{name}-%{version}.
So %{_builddir} / %{buildsubdir} will be ie. /usr/src/rpm/BUILD / foo-1.2.3.

 Meanwhile, I'm not sure what you have discovered: literally nothing has
 changed in this area for most of this century, the whole mechanism
 if fabulously broken and mis-designed and mis-implemented imho.
Simply that $PWD == %{_builddir}/%{?buildsubdir}, thus $RPM_BUILD_DIR=`pwd`
is incorrect with the consequence of messing up paths in debug packages..

--
Regards,
Per Øyvind
__
RPM Package Managerhttp://rpm5.org
Developer Communication Listrpm-devel@rpm5.org