Skip to content

[SYCL] Omit extensions (.a .o .out .lib .obj .exe) in output project … #4250

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.

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

Merged
merged 2 commits into from
Aug 6, 2021

Conversation

qichaogu
Copy link
Contributor

@qichaogu qichaogu commented Aug 4, 2021

…directory name

For "-o file.xxx" where the option value has an extension, if the extension is one of .a .o .out .lib .obj .exe, the output project directory name will be file.proj which omits the extension. Otherwise the output project directory name will be file.xxx.prj which keeps the original extension.

Signed-off-by: Qichao Gu [email protected]

@qichaogu qichaogu force-pushed the output-project-dir-name branch 2 times, most recently from 3fc99ec to 2fcfb10 Compare August 4, 2021 03:25
mdtoguchi
mdtoguchi previously approved these changes Aug 4, 2021
@qichaogu qichaogu force-pushed the output-project-dir-name branch 2 times, most recently from 1b9f036 to 42efd79 Compare August 5, 2021 01:10
…directory name

For "-o file.xxx" where the option value has an extension, if the
extension is one of .a .o .out .lib .obj .exe, the output project
directory name will be file.proj which omits the extension. Otherwise
the output project directory name will be file.xxx.prj which keeps
the original extension.

Signed-off-by: Qichao Gu <[email protected]>
@qichaogu qichaogu force-pushed the output-project-dir-name branch from 42efd79 to b98bb06 Compare August 5, 2021 01:13
mdtoguchi
mdtoguchi previously approved these changes Aug 5, 2021
@keryell
Copy link
Contributor

keryell commented Aug 5, 2021

You should adapt the PR description as a comment in the code.
A reader of the code has no clue about the "why"...

…directory name

For "-o file.xxx" where the option value has an extension, if the
extension is one of .a .o .out .lib .obj .exe, the output project
directory name will be file.proj which omits the extension. Otherwise
the output project directory name will be file.xxx.prj which keeps
the original extension.

Signed-off-by: Qichao Gu <[email protected]>
@qichaogu
Copy link
Contributor Author

qichaogu commented Aug 6, 2021

You should adapt the PR description as a comment in the code.
A reader of the code has no clue about the "why"...

Done.

Copy link
Contributor

@keryell keryell left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the clarification!

@bader bader merged commit d8237a6 into intel:sycl Aug 6, 2021
@qichaogu qichaogu deleted the output-project-dir-name branch August 11, 2021 01:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants