Project

General

Profile

Bug #4280

sys/ipmi.h should be in system/header package

Added by Marcel Telka about 7 years ago. Updated about 7 years ago.

Status:
Rejected
Priority:
Normal
Assignee:
Category:
driver - device drivers
Start date:
2013-10-31
Due date:
% Done:

0%

Estimated time:
Difficulty:
Bite-size
Tags:
needs-triage
Gerrit CR:

Description

The sys/ipmi.h file should be moved from driver/ipmi package to system/header package. The rationale is simple: I want to compile IPMI-aware sources even on a machine without IPMI support (and without the driver/ipmi package installed).

#1

Updated by Marcel Telka about 7 years ago

  • Status changed from In Progress to Feedback
  • Assignee deleted (Marcel Telka)

Further analysis showed that this is common. For example /usr/include/sys/stmf.h is in storage/stmf package. So, I'll leave this as is.

This bug should be closed.

#2

Updated by Marcel Telka about 7 years ago

  • Status changed from Feedback to Rejected
#3

Updated by Yuri Pankov about 7 years ago

Common doesn't mean correct, I actually liked the idea.

#4

Updated by Marcel Telka about 7 years ago

  • Status changed from Rejected to Feedback
  • Assignee set to Marcel Telka

I agree. Especially when I found that /usr/include/libipmi.h is in system/header too.

#5

Updated by Marcel Telka about 7 years ago

  • Status changed from Feedback to In Progress
#6

Updated by Rich Lowe about 7 years ago

In general, we've been trying to move stuff out of system/header, into the packages that actually deliver the pieces. Massive packages that are dumping grounds for massive amounts of unrelated stuff have, well, been bad.

So the history is that we used to just cram everything into as few packages as possible, then we decided to actual package all the parts of a component in the same place.

I don't have any opinions on the specifics, beyond not really understanding why it's worth it just to avoid the installation of one, small, well-behaved, package.

#7

Updated by Marcel Telka about 7 years ago

  • Status changed from In Progress to Feedback

Okay, that makes sense.

#8

Updated by Marcel Telka about 7 years ago

  • Status changed from Feedback to Rejected

Also available in: Atom PDF