https://bugzilla.novell.com/show_bug.cgi?id=714995
https://bugzilla.novell.com/show_bug.cgi?id=714995#c17
--- Comment #17 from Dennis Olsson 2011-11-04 10:55:08 CET ---
Reply to comment #15:
Yes, I know - that was why I took the time to review / test / rewrite it.
(Although, I have to admit, the real reason was that I did not really like the
code, I was reading during the code review;-)
Reply to comment #16:
Hhhnmmmm, you are making a valid point.
I originally added this test to Thomas' version of the patch, while it was
throwing errors, when no iSCSI attachments were active, which is often the case
when running the script manually during testing of the iSCSI setup.
But, since then, I have completely rewritten the patch, so that this test is
not strictly necessary any more - at least not for the original reasons.-)
Now, it is only an optimization to avoid running through the entire mount
points in case no iSCSI devices have been attached.
So, strictly speaking the test is not necessary, but does come in handy, when
running the script manually during testings.-) Have thus left it there.
--
Configure bugmail: https://bugzilla.novell.com/userprefs.cgi?tab=email
------- You are receiving this mail because: -------
You are on the CC list for the bug.