Bug 1172414 - Review Request: ltp - Linux Test Project
Summary: Review Request: ltp - Linux Test Project
Keywords:
Status: CLOSED NOTABUG
Alias: None
Product: Fedora
Classification: Fedora
Component: Package Review
Version: rawhide
Hardware: All
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Nobody's working on this, feel free to take it
QA Contact: Fedora Extras Quality Assurance
URL:
Whiteboard:
Depends On:
Blocks: FE-DEADREVIEW
TreeView+ depends on / blocked
 
Reported: 2014-12-10 01:59 UTC by Ken Dreyer
Modified: 2021-08-27 00:45 UTC (History)
4 users (show)

Fixed In Version:
Clone Of:
Environment:
Last Closed: 2021-08-27 00:45:40 UTC
Type: ---
Embargoed:


Attachments (Terms of Use)

Description Ken Dreyer 2014-12-10 01:59:37 UTC
Spec URL: https://ktdreyer.fedorapeople.org/reviews/ltp.spec
SRPM URL: https://ktdreyer.fedorapeople.org/reviews/ltp-20140828-1.fc22.src.rpm
Description: The LTP testsuite contains a collection of tools for testing the Linux kernel and related features. Our goal is to improve the Linux kernel and system libraries by bringing test automation to the testing effort.
Fedora Account System Username: ktdreyer

Rawhide (F22) scratch build: http://koji.fedoraproject.org/koji/taskinfo?taskID=8333398

Comment 1 Mamoru TASAKA 2014-12-10 09:54:51 UTC
Some notes:

- Licensing

Well, this package is a bit hard because the source
tarball contains 8119 files, with 3 of them are .tar.gz
which contains another 49 files.

From the point of view with regard to licensing check
my first "grep"ing classified 6115 files, I had to check
the rest ~2000 files with my eyes. While I may have overlooked
something, one thing I detected:

* APSL 1.1 
  - not sure for 1.1, however APSL 2.0 is marked as
    incompatible with GPL 
testcases/kernel/fs/fsx-linux/fsx-linux.c

- Debuginfo issue
I think using
%undefine _missing_build_ids_terminate_build
is better than removing executable permission from
.obj files (not linked, missing build-id).

Full review follows. By the way, I would appreciate it
if you would review my review request (bug 1171991)

Comment 2 Mamoru TASAKA 2014-12-11 09:26:04 UTC
Well,

* Documentation
  - At least please add COPYING as %license
  - Also, please consider to add some files to %doc, such
    as README.

* Licensing issue
  - See above

* Debuginfo issue
  - See above

* rpmlint
  - Well, while I ignore "missing-call-to-setgroups-before-setuid"
    rpmlint issue (because these files are for testing such
    syscalls), now:

  * Lots of "non-standard-executable-perm 0775L" rpmlints
    - It should be okay with 0755 (as owner/group is root:root)

  * Some "script-without-shebang" rpmlints
    - As these does not have shebang (and at least some of files
      look like just a text file), these need not have executable
      permission
      (or is this intentional?)

  * Some "spurious-executable-perm" error on testcases/bin/*.c
    - While I am not sure if these *.c files are really needed
      (in the binary rpm), at least executable permissions should
      not be necessary.
      (note that also "devel-file-in-non-devel-package" rpmlint
      appears)

Comment 3 Mamoru TASAKA 2015-01-23 08:11:54 UTC
Would you update the status?

Comment 4 Ken Dreyer 2015-01-27 06:03:51 UTC
Thanks for your comments, and apologies for the delay. Here's a new version that incorporates many of the changes you've mentioned.

* Wed Jan 07 2015 Ken Dreyer <ktdreyer> - 20150119-1
- Update to latest upstream version
- Drop upstreamed patches, and add more for format-string bugs
- Use _missing_build_ids_terminate_build macro (RHBZ #1172414)
- Include COPYING and README (RHBZ #1172414)
- Fix group-writable permissions for rpmlint (RHBZ #1172414)

Exact changes in Git: https://fedorapeople.org/cgit/ktdreyer/public_git/ltp.git/commit/?id=fefba6a3b1efd01e5c1a43447f8607367188d940

Spec URL: https://ktdreyer.fedorapeople.org/reviews/ltp.spec
SRPM URL: https://ktdreyer.fedorapeople.org/reviews/ltp-20150119-1.fc22.src.rpm

I'm still working on all the rpmlint changes - those are taking me some time to fix.

Comment 5 Mamoru TASAKA 2015-01-30 10:06:15 UTC
Okay, now only rpmlint issue is left.

For now I can ignore
- missing-call-to-setgroups-before-setuid
- unstripped-binary-or-object (for .obj - perhaps this cannot be avoided)
- incorrect-fsf-address
- no-manual-page-for-binary
- spelling-error
- manual-page-warning
- missing-call-to-chdir-with-chroot
- uncompressed-zip

So
  - I cannot figure out if the files listed as "devel-file-in-non-devel-package"
    is really needed.
  - And lots of permission issue (script-without-shebang)
    Some idea of mine
    - find out files with executable permission 
      (find %{buildroot} -type f -perm /1000)
    - If it is binary, for now ignore
    - If it is text, check if the text has shebang or not
    - If not having shebang, then drop executable permission

Comment 6 Upstream Release Monitoring 2015-12-06 18:26:09 UTC
pbrobinson's scratch build of linux-user-chroot?#b7afe5173cbd31b029b027b6f8a14baa5e6ce87a for epel7-archbootstrap and git://pkgs.fedoraproject.org/linux-user-chroot?#b7afe5173cbd31b029b027b6f8a14baa5e6ce87a failed http://koji.fedoraproject.org/koji/taskinfo?taskID=12089939

Comment 7 Package Review 2020-07-10 00:51:04 UTC
This is an automatic check from review-stats script.

This review request ticket hasn't been updated for some time, but it seems
that the review is still being working out by you. If this is right, please
respond to this comment clearing the NEEDINFO flag and try to reach out the
submitter to proceed with the review.

If you're not interested in reviewing this ticket anymore, please clear the
fedora-review flag and reset the assignee, so that a new reviewer can take
this ticket.

Without any reply, this request will shortly be resetted.

Comment 8 Package Review 2020-11-13 00:46:22 UTC
This is an automatic action taken by review-stats script.

The ticket reviewer failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we reset the status and the assignee of this ticket.

Comment 9 Package Review 2021-08-27 00:45:40 UTC
This is an automatic action taken by review-stats script.

The ticket submitter failed to clear the NEEDINFO flag in a month.
As per https://fedoraproject.org/wiki/Policy_for_stalled_package_reviews
we consider this ticket as DEADREVIEW and proceed to close it.


Note You need to log in before you can comment on or make changes to this bug.