Fixing wrong Bootstrapping part in Guidelines

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Dear Packagers who are using Boostrapping logic for the cyclical dependency

Need your help to fix wrong Bootstrapping part in Guidelines.
This mail is long.
Sorry for that in advance.

You may be building the cyclical dependency packages by using a variable such as _with_bootstrap, need_bootstrap, bootstrap, enable_test, and etc..
For example, you may build with below ways for that, if you will use mock command.

```
$ mock -r fedora-rawhide-x86_64 --with=bootstrap *.src.rpm
  => _with_bootstrap can be used as --with=bootstrap
$ mock -r fedora-rawhide-x86_64 --define '_with_bootstrap 1' *.src.rpm
$ mock -r fedora-rawhide-x86_64 --define 'need_bootstrap 1' *.src.rpm
$ mock -r fedora-rawhide-x86_64 --define 'enable_test 1' *.src.rpm
...
```

Here is a document page to unify the Bootstrap logic.
You may know it.
https://fedoraproject.org/wiki/Packaging:Guidelines#Bootstrapping

However I found that below part in the page is wrong.

```
%{!?_with_bootstrap: %global bootstrap 1}
```

Because ..
If _with_bootstrap is not set from outside, bootstrap is 1
  => bootstrap is True/Enabled
if _with_bootstrap is set as 1 from outside, bootstrap's value is not set.
  => the value is empty if it is not declared in advance. It's a kind of 0. bootstrap is False/Disabled.

This situation is opposite meaning of "_with_bootstrap".

Below way not using negative operator `!?` is correct.

```
%{?_with_bootstrap: %global bootstrap 1}
```


The reason why I wrote this here is 

I found that had already been reported 2 years ago for packaging committee, however it was closed without fixing.

https://pagure.io/packaging-committee/issue/509


I am not sure that why it is not admitted. 

You may feel that it does not matter because you may edit the Bootstrapping logic in the RPM spec file manually.

But in my case, I am one of the people who use the Bootsrapping logic actively.
There are 89 RPM packages that constitutes Ruby on Rails 5.0.
To build Ruby on Rails 5.0 completely from scratch, I have to build the packages total 103 times considering bootstrap.

I am trying to build those packages automatically by a tool [1] with a configuration file [2] for Ruby on Rails.
It is important to fix it due to that.

Fortunately today another guy Vit created new ticket for that.
So, if YOU like this proposal, please comment in below page of the ticket or reply here.
It is helpful for us to move this huge rock. I really want to fix it.

"I like it." comment please.
=> https://pagure.io/packaging-committee/issue/684


Thank you for your help.

[1] https://github.com/sclorg/rpm-list-builder
[2] https://github.com/sclorg/rhscl-rebuild-recipes/blob/master/ror.yml


Kind regards,
Jun Aruga

_______________________________________________
packaging mailing list -- packaging@xxxxxxxxxxxxxxxxxxxxxxx
To unsubscribe send an email to packaging-leave@xxxxxxxxxxxxxxxxxxxxxxx




[Index of Archives]     [Fedora Users]     [Fedora Desktop]     [Fedora SELinux]     [Big List of Linux Books]     [Yosemite Forum]     [KDE Users]

  Powered by Linux