View Issue Details
| ID | Project | Category | View Status | Date Submitted | Last Update |
|---|---|---|---|---|---|
| 0001734 | Issue 8 drafts | Shell and Utilities | public | 2023-05-31 16:54 | 2024-06-11 09:12 |
| Reporter | ajosey | Assigned To | |||
| Priority | normal | Severity | Objection | Type | Error |
| Status | Closed | Resolution | Accepted As Marked | ||
| Product Version | Draft 3 | ||||
| Name | Andrew Josey | ||||
| Organization | |||||
| User Reference | IEEE/I-2 | ||||
| Section | mkdtemp | ||||
| Page Number | 1410 | ||||
| Line Number | 47358 | ||||
| Final Accepted Text | See 0001734:0006303 | ||||
| Summary | 0001734: XSH mkdtemp function errors (IEEE/I-2) | ||||
| Description | The EILSEQ error is in the mkdtemp() shall fail section of the errors, but all three of the functions in this section have a template argument and the same problem should be required to be detected in the mkostemp() and mkstemp() functions as well as in the mkdtemp() function. There is also a missing period at the end of this error. | ||||
| Desired Action | Move P1410, L47358-47359 to appear after L47353 and add a period to the end of the moved text. | ||||
| Tags | applied_after_i8d3, issue8 | ||||
|
|
The EILSEQ error for mkostemp() and mkstemp(), along with almost all errors for those two functions, are covered by this line (47381) at the end of ERRORS:Additional error conditions for the mkstemp() and mkostemp() functions are defined in open(). The only errors for mkostemp() and mkstemp() that need to be stated separately on the mkdtemp() page are those which do not derive from the requirement (line 47335) that they create the file, and obtain a file descriptor for it, as if by a call to: open(pathname, O_RDWR|O_CREAT|O_EXCL, S_IRUSR|S_IWUSR) There is also a period at the end of the EILSEQ description - it is not missing. |
|
|
Re 0001734:0006297: Geoff, you're correct on both of your points. Looking more closely at this, I should have noted that the error handling for mkdtemp() should be handled the same way as the errors for mkostemp() and mkstemp(). And there is a missing period at the end of an EILSEQ error, but it is on the mkdir() page instead of on the mkdtemp() page. Consider replacing the entire Desired Action to the following: Add a period to the end of the EILSEQ error condition for mkdir() on P1405, L47222. Delete P1410, L47355-47378 (ERRORS for mkdtemp()). Add a new sentence to the start of the paragraph on P1410, L47381 (ERRORS for mkdtemp()): Additional error conditions for the mkdtemp( ) function are defined in mkdir( ). |
|
|
Re 0001734:0006303, Good point - I agree all of the mkdtemp()-specific errors should be handled by reference to mkdir() and your new proposed changes will do that. You're also right about the missing period for mkdir() EILSEQ, but note L47222 is on P1405 not P1406. |
|
|
0001734:0006303 has been updated to correct the page number error noted in 0001734:0006304. |
| Date Modified | Username | Field | Change |
|---|---|---|---|
| 2023-05-31 16:54 | ajosey | New Issue | |
| 2023-05-31 16:54 | ajosey | Name | => Andrew Josey |
| 2023-05-31 16:54 | ajosey | User Reference | => IEEE/I-2 |
| 2023-05-31 16:54 | ajosey | Section | => mkdtemp |
| 2023-05-31 16:54 | ajosey | Page Number | => 1410 |
| 2023-05-31 16:54 | ajosey | Line Number | => 47358 |
| 2023-05-31 17:14 | ajosey | Summary | XSH mkdtemp function errors => XSH mkdtemp function errors (IEEE/I-2) |
| 2023-06-01 09:24 | geoffclare | Note Added: 0006297 | |
| 2023-06-01 17:51 | Don Cragun | Note Added: 0006303 | |
| 2023-06-02 08:52 | geoffclare | Note Added: 0006304 | |
| 2023-06-02 13:26 | Don Cragun | Note Edited: 0006303 | |
| 2023-06-02 13:27 | Don Cragun | Note Added: 0006305 | |
| 2023-07-13 15:26 | nick | Final Accepted Text | => See 0001734:0006303 |
| 2023-07-13 15:26 | nick | Status | New => Resolved |
| 2023-07-13 15:26 | nick | Resolution | Open => Accepted As Marked |
| 2023-07-13 15:27 | nick | Tag Attached: issue8 | |
| 2023-08-08 11:19 | geoffclare | Status | Resolved => Applied |
| 2023-08-08 11:20 | geoffclare | Tag Attached: applied_after_i8d3 | |
| 2024-06-11 09:12 | agadmin | Status | Applied => Closed |
