Bug 118394 - W8 C6 exercise grading too restrictive...
Summary: W8 C6 exercise grading too restrictive...
Keywords:
Status: CLOSED CURRENTRELEASE
Alias: None
Product: Red Hat Academy
Classification: Retired
Component: RHA030 Curriculum
Version: 2.0
Hardware: i386
OS: Linux
medium
medium
Target Milestone: ---
Assignee: Bowe Strickland
QA Contact: Bowe Strickland
URL:
Whiteboard:
: 122702 (view as bug list)
Depends On:
Blocks:
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:
Clone Of:
Environment:
Last Closed: 2005-01-07 17:30:14 UTC
Embargoed:


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
/usr/share/gedit-2/taglist/Latex.tags/home/ql004/gedit-2/taglist/Latex.tagsnot


this, which is more common (note the order of switches):diff -ru
/usr/share/gedit-2/taglist/Latex.tags/home/ql004/gedit-2/taglist/Latex.tags

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.