Bug 118394 - W8 C6 exercise grading too restrictive...
Summary: W8 C6 exercise grading too restrictive...
Alias: None
Product: Red Hat Academy
Classification: Retired
Component: RHA030 Curriculum   
(Show other bugs)
Version: 2.0
Hardware: i386 Linux
Target Milestone: ---
Assignee: Bowe Strickland
QA Contact: Bowe Strickland
: 122702 (view as bug list)
Depends On:
TreeView+ depends on / blocked
Reported: 2004-03-16 11:54 UTC by Bowe Strickland
Modified: 2007-04-18 17:04 UTC (History)
2 users (show)

Fixed In Version:
Doc Type: Bug Fix
Doc Text:
Story Points: ---
Clone Of:
Last Closed: 2005-01-07 17:30:14 UTC
Type: ---
Regression: ---
Mount Type: ---
Documentation: ---
Verified Versions:
Category: ---
oVirt Team: ---
RHEL 7.3 requirements from Atomic Host:
Cloudforms Team: ---

Attachments (Terms of Use)

Description Bowe Strickland 2004-03-16 11:54:42 UTC
Apparently, the grading is only accepting the results only if
thiscommand is run:

diff -ur

this, which is more common (note the order of switches):diff -ru

Comment 1 Garry 2004-05-10 06:05:32 UTC
*** Bug 122702 has been marked as a duplicate of this bug. ***

Comment 2 Brad Smith 2004-06-15 02:30:05 UTC
The reason this fails is because diff -r always includes the complete
command-line used to generate each comparison. Thus the outputs of
diff -ur and diff -ru will always be different though never in a
significant way. 

Possible solutions:

 * Alter the back-end to diff the result of `grep -v ^diff filename`
instead of filename (this should not result in any false positives
since anything from the actual text of the documents should be
preceeded with '+++' or '---'. 

 * Alter the exercise to do a diff -u on two individual files instead
of on a directory tree. 

 * Alter the exercise to do a diff -r (without other arguments) on the
directory trees.

Bowe and I will work on this and update the courseware as nescessary.

Comment 3 Brad Smith 2004-06-17 23:00:37 UTC
Fix is in CVS.

The simplest solution was chosen: The exercise now has the student do
a recursive diff instead of a unified, recursive diff.

Comment 4 Bowe Strickland 2005-01-05 16:33:29 UTC
updated script to match text

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