[Date Prev][Date Next] [Thread Prev][Thread Next] [Date Index] [Thread Index]

Re: hardinfo status.



tags 385781 moreinfo
thanks

On Thu, Sep 07, 2006 at 03:19:57AM -0700, Steve Langasek wrote:
> On Wed, Sep 06, 2006 at 02:38:17PM -0300, Gustavo Franco wrote:
> > >0.3.7pre-4, however, did build on alpha, arm, ia64, m68k and s390, and the
> > >corresponding binaries are still in unstable. The maintainer has already
> > >requested their removal (#385781) a few days ago, so it's currently waiting
> > >for that to be actioned.
> 
> > In other words, the real status is that there are old binaries in
> > architectures that aren't listed in hardinfo anymore and once #385781
> > is closed, it will be able to move into testing, right?
> 
> It's correct that as long as 385781 is open, the package isn't going to
> migrate into testing.
> 
> But as an alpha porter I object to 385781 being closed without there having
> been any consultation of debian-alpha.  I don't see any reason that
> architectures such as alpha, arm, and ia64 that clearly have the interfaces
> hardinfo is intended to apply to should suddenly lose support for it as a
> consequence of dubious upstream code abstractions.  I.e., the *only*
> differences between any of the existing arch/linux/foo trees in this package
> are in the processor.h file, which is itself unpleasantly non-headerlike,
> consisting entirely of static function definitions.
> 
> I would probably be more than happy to write up processor_get_info() and
> computer_get_processor() routines for alpha, but the additional busywork
> imposed on porters by the poor overall code layout bothers me.  What could
> have been a single build-time define and a C file with #ifdef'ed
> architecture-specific routines is instead a symlink farm requiring manual
> porting work just to get the package to recognize a new architecture
> name(!).  Is this really an improvement over the portable code currently in
> testing, for any architecture?
> 
> BTW, the sparc binaries are silently broken:
> 
> In file included from computer.c:70:
> ./arch/this/processor.h: In function 'computer_get_processor':
> ./arch/this/processor.h:44: warning: implicit declaration of function 'get_processor_strfamily'
> mv -f computer.so modules
> 
> http://buildd.debian.org/fetch.php?&pkg=hardinfo&ver=0.4.1-2&arch=sparc&stamp=1151034855&file=log&as=raw

Agney, please comment on this mail.

--Jeroen

-- 
Jeroen van Wolffelaar
Jeroen@wolffelaar.nl (also for Jabber & MSN; ICQ: 33944357)
http://Jeroen.A-Eskwadraat.nl



Reply to: