[Bug 226704] Review Request: iasl - Intel acpi compiler

[bugzilla at redhat.com]

Please do not reply directly to this email. All additional
comments should be made in the comments box of this bug report.

Summary: Review Request: iasl - Intel acpi compiler


https://bugzilla.redhat.com/bugzilla/show_bug.cgi?id=226704





------- Additional Comments From opensource at till.name  2007-02-20 13:40 EST -------
Spec URL: http://www-users.kawo2.rwth-aachen.de/~tmaas/fedora/iasl.spec
SRPM URL:
http://www-users.kawo2.rwth-aachen.de/~tmaas/fedora/repo/iasl-20061109-2.src.rpm

(In reply to comment #3)
> rpmlint:
> E: iasl description-line-too-long iasl compiles ASL (ACPI Source Language) into
> AML (ACPI Machine Language), which
> - reformat to use less than 70 chars, I guess
Fixed, has to be less  that 80 chars, it was excactly 80 chars.

> W: iasl invalid-license Intel Software License Agreement

See the comment in the spec file:
https://www.redhat.com/archives/fedora-extras-list/2007-January/msg00427.html

> W: iasl spurious-executable-perm /usr/share/man/man1/iasl.1.gz
> - chmod 0644

fixed

> I would also package the other utilities if possible.

Do you want to do it or do you use the utitilities? I am not very keen to
package them unless they are very needed. If you want to maintain iasl, you can
have it. Actually I only wanted to include it to Extras because it is needed by
virtualbox, but since I found out that virtualbox needs at least 2 more packages
in fedora it exceeds the amount of work that I have free for this. So if you do
not need iasl or the other contents of the tarball or want to maintain it, I
will close this ticket, soon.

-- 
Configure bugmail: https://bugzilla.redhat.com/bugzilla/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are the QA contact for the bug, or are watching the QA contact.