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.

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

Allow repository name change via git scm basename paramter #479

Open
wants to merge 1 commit into
base: master
Choose a base branch
from

Conversation

watologo1
Copy link

Example:
https://github.com/DMTF/Redfishtool.git
vs
network:utilities/redfishtool

In mainline project the first letter 'R' is
uppercase, but in OBS it's lowercase.

Adding basename= parameter in _service file with this patch like:
main
@PARENT_TAG@.@[email protected]%h
enable

  • redfishtool

results in:
redfishtool
redfishtool-1.1.7.1.gfaa99dc.obscpio
redfishtool-1.1.7.1.gfaa99dc.tar.xz
redfishtool.obsinfo

instead of (without basename param):
Redfishtool
Redfishtool-1.1.7.1.gfaa99dc.obscpio
Redfishtool-1.1.7.1.gfaa99dc.tar.xz
Redfishtool.obsinfo

With this patch it is possible to adopt or rename
any arbitrary git project name to its matching OBS project name and process it via OBS services.

TarSCM/cli.py Outdated
parser.add_argument('--basename', default = None,
help='Checkout/rename and further process repository as '
'<BASENAME> (requires git scm).'
'Make sure there are no leftovers (workrepo, obsinfo'

Choose a reason for hiding this comment

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

line too long (81 > 79 characters)

TarSCM/cli.py Outdated
@@ -194,6 +195,11 @@ def parse_args(self, options):
'(only used with \'--latest-signed-*\')')
parser.add_argument('--without-version', default = False,
help='Do not add version to output file.')
parser.add_argument('--basename', default = None,
help='Checkout/rename and further process repository as '

Choose a reason for hiding this comment

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

line too long (85 > 79 characters)

TarSCM/cli.py Outdated
@@ -194,6 +195,11 @@ def parse_args(self, options):
'(only used with \'--latest-signed-*\')')
parser.add_argument('--without-version', default = False,
help='Do not add version to output file.')
parser.add_argument('--basename', default = None,
help='Checkout/rename and further process repository '

Choose a reason for hiding this comment

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

line too long (82 > 79 characters)

Example:
https://github.com/DMTF/Redfishtool.git
vs
network:utilities/redfishtool

In mainline project the first letter 'R' is
uppercase, but in OBS it's lowercase.

Adding basename= parameter in _service file with this patch like:
     <param name="revision">main</param>
     <param name="versionformat">@PARENT_TAG@.@[email protected]%h</param>
     <param name="changesgenerate">enable</param>
-    </service>
+    <param name="basename">redfishtool</param>
+  </service>
   <service name="set_version" mode="localonly"/>
   <service name="tar" mode="buildtime">
   </service>

results in:
redfishtool
redfishtool-1.1.7.1.gfaa99dc.obscpio
redfishtool-1.1.7.1.gfaa99dc.tar.xz
redfishtool.obsinfo

instead of (without basename param):
Redfishtool
Redfishtool-1.1.7.1.gfaa99dc.obscpio
Redfishtool-1.1.7.1.gfaa99dc.tar.xz
Redfishtool.obsinfo

With this patch it is possible to adopt or rename
any arbitrary git project name to its matching OBS project
name and process it via OBS services.
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.

2 participants