[Bug 21876] New: Untracked changes prevent building
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21876 Bug ID: 21876 Summary: Untracked changes prevent building Change sponsored?: --- Product: Koha Version: master Hardware: All OS: All Status: NEW Severity: minor Priority: P5 - low Component: Packaging Assignee: koha-bugs@lists.koha-community.org Reporter: mtompset@hotmail.com QA Contact: testopia@bugs.koha-community.org CC: mirko@abunchofthings.net On bug 18993 comment #29, I mentioned difficulty following the instructions to package up Koha following the instructions given at https://wiki.koha-community.org/wiki/Building_Debian_Packages_-_The_Easy_Way With tcohen's patience and help, I believe I determined my problem was untracked files. Add functionality to check for untracked files to the build command. -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21876 M. Tompsett <mtompset@hotmail.com> changed: What |Removed |Added ---------------------------------------------------------------------------- Assignee|koha-bugs@lists.koha-commun |mtompset@hotmail.com |ity.org | Status|NEW |ASSIGNED CC| |mtompset@hotmail.com -- You are receiving this mail because: You are watching all bug changes. You are the assignee for the bug.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21876 --- Comment #1 from M. Tompsett <mtompset@hotmail.com> --- Turns out the everything_is_committed function is not working. grep /^xxx/ is not working as expected. Minor tweak should fix that, but I'm also going to improve the error message. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21876 --- Comment #2 from M. Tompsett <mtompset@hotmail.com> --- Created attachment 82620 --> https://bugs.koha-community.org/bugzilla3/attachment.cgi?id=82620&action=edit Bug 21876: tweak the build script While trying to build following the instructions on the wiki, error after error was encountered, and frustration rose. Eventually, tcohen mentioned untracked files. So, all untracked files were purposefully deleted all, and then building succeeded again. To this end, the build script wa tweaked. 1) Sorry, a bunch of perlcritic things to get it perlcritic -2 clear were done. :) Proof: perlcritic -1 debian/build-git-snapshot -- fear the long scrolly output apply the patch repeat -- much shorter, and who cares about the print return value! perlcritic -2 debian/build-git-snapshot -- OK. YAY! 2) everything_is_committed was broken. It would always return 1 and keep going. Proof: touch foobar. add a blank line to Koha.pm now try to build. It will spend minutes and die miserably at the end. To address this, it was tweaked to check for two cases: uncommitted and untracked. You can test this with a combination of touch foobar and blank line to Koha.pm to see. 3) While not a huge issue, the version should be better determined rather than hard coded as 16.06~git. Logic was added to better determine something based on Koha.pm through grep'ing, so as to not need PERL5LIB set. Proof: Try checking out 17.11.x apply this patch (hopefully it does) try to build without a -v parameter. -- it should generate 17.11~git Feel free to repeat with 18.05 and master they should generate 18.05~git and 18.06~git As always run the qa test tools. ;) Oh wait, they don't run unless you rename this to a .pl file. I guess that's another bug. And if you do rename to test, it should only complain about Robin's last name. -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21876 --- Comment #3 from M. Tompsett <mtompset@hotmail.com> --- Oh, and before I forget, I purposefully installed dh-make and got libwebservice-ils-perl installed in the pbuilder while testing. That may be an important substep that isn't documented. :) -- You are receiving this mail because: You are watching all bug changes.
https://bugs.koha-community.org/bugzilla3/show_bug.cgi?id=21876 Katrin Fischer <katrin.fischer@bsz-bw.de> changed: What |Removed |Added ---------------------------------------------------------------------------- CC| |katrin.fischer@bsz-bw.de --- Comment #4 from Katrin Fischer <katrin.fischer@bsz-bw.de> --- Should this be needs sign-off? -- You are receiving this mail because: You are watching all bug changes.
participants (1)
-
bugzilla-daemon@bugs.koha-community.org