Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

time.1: alignment and standards nit + spdx #1315

Closed
wants to merge 1 commit into from
Closed

time.1: alignment and standards nit + spdx #1315

wants to merge 1 commit into from

Conversation

concussious
Copy link
Contributor

  • spdx tagged
  • options spacing was one char short
  • example file was 29 char too long
  • use standard macro for posix.2 spec

Is the commit log okay?
I don't want to bloat it.

@concussious concussious marked this pull request as ready for review July 5, 2024 15:30
@concussious
time.1: minor cleanup (alignment/macro/spdx)
1acb3e9 
+ shorter example filename to minimize line wrap
+ standards macro clarifying posix => posix.2
+ align options + tag spdx

MFC after:	3 days
Pull-Request:	freebsd#1315
mhorne pushed a commit to mhorne/freebsd that referenced this pull request Jul 12, 2024
@concussious @mhorne
time.1: minor cleanup (alignment/macro/spdx)
cb18ba9 
+ shorter example filename to minimize line wrap
+ standards macro clarifying posix => posix.2
+ align options + tag spdx

Reviewed by:	mhorne
MFC after:	3 days
Pull-Request:	freebsd#1315
@mhorne
Copy link
Contributor

mhorne commented Jul 12, 2024

Merged to main as cb18ba9. This PR can be closed.

@concussious
Copy link
Contributor Author

Thanks!

bsdjhb pushed a commit to bsdjhb/cheribsd that referenced this pull request Oct 30, 2024
@concussious @bsdjhb
time.1: minor cleanup (alignment/macro/spdx)
88c747f 
+ shorter example filename to minimize line wrap
+ standards macro clarifying posix => posix.2
+ align options + tag spdx

Reviewed by:	mhorne
MFC after:	3 days
Pull-Request:	freebsd/freebsd-src#1315
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@mhorne mhorne mhorne approved these changes

Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.
2 participants
@concussious @mhorne