Skip to content

Conversation

@xambroz
Copy link

@xambroz xambroz commented Feb 19, 2017

Hello,
I have done some updates to the spec teplate for Fedora.
On one hand Fedora wants to prefer python3 packages over python2, on the other hand it is not always possible.
I have tried to make the package compliant with the current Fedora python packaging guidelines while keeping easy define to switch off the python3 generation.
Best regards
Michal Ambroz

@xambroz xambroz mentioned this pull request Feb 20, 2017
data['source_url'] = args.name + '-' + args.version + '.zip'
data['year'] = datetime.datetime.now().year # set current year

now = datetime.datetime.now()
Copy link
Member

Choose a reason for hiding this comment

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

Why did you change this? AFAICS you never use it in the fedora template..

Copy link
Author

Choose a reason for hiding this comment

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

I didn't feel like calling now() twice for no particular reason.

The 'year' was already initialized by the existing init ... that is not changed. (there is data['year'] = now.year)

I am adding template of 'changelog_date' in a format suitable for Fedora changes.
The 'changelog_date' is used in the spec file template.

@toabctl
Copy link
Member

toabctl commented Mar 2, 2017

@xambroz thanks for the PR! Is there a easy way (without a Fedora system) to test this somehow?

@toabctl
Copy link
Member

toabctl commented May 8, 2017

@xambroz friendly ping...

Copy link
Author

@xambroz xambroz left a comment

Choose a reason for hiding this comment

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

Is there a easy way (without a Fedora system) to test this somehow?

@toabctl Best way testing the spec file is of course on Fedora. Generating the spec file and trying to build that one with rpmbuild.

Would you be having some easy way testing spec file for SuSE without SuSE ?

data['source_url'] = args.name + '-' + args.version + '.zip'
data['year'] = datetime.datetime.now().year # set current year

now = datetime.datetime.now()
Copy link
Author

Choose a reason for hiding this comment

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

I didn't feel like calling now() twice for no particular reason.

The 'year' was already initialized by the existing init ... that is not changed. (there is data['year'] = now.year)

I am adding template of 'changelog_date' in a format suitable for Fedora changes.
The 'changelog_date' is used in the spec file template.

@nkadel
Copy link

nkadel commented Oct 18, 2018

There's a tool called "mock" that I use on Fedora and RHEL to test builds across multiple platforms. I see that it has built-in support for the OpenSuSE "tumbleweed" release, which may be helpful for multiple platform testing. as asked by xambroz.

@mcepl
Copy link
Contributor

mcepl commented May 10, 2022

@xambroz Does this still make sense? If yes, could you rebase it, please?

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants