[New: openFATE 311072] Turn quilt into a noarch package
Feature added by: Jean Delvare (jdelvare) Feature #311072, revision 1 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 7.5 % of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 2 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) + Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 7.5 % of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 3 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 7.5 % of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. + Business case (Partner benefit): + openSUSE.org: This change will improve the portability of quilt by + limiting the build requirements. + This change will improve the build time of quilt in OBS. + This change will make it easier to contribute to quilt, by lowering the + skills required. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Klaas Freitag (kfreitag) Feature #311072, revision 4 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 7.5 % of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. + Discussion: + #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) + I think this is a bug report rather than a hackweek project. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 5 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 7.5 % of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. + #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) + I'm not quite sure why you say that. Quilt works as is, so we can't + call it a bug. This is only a bizarre design to mix programming + language when you really don't have to. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 6 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) + Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 7.5 % of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 7 Title: Turn quilt into a noarch package Hackweek VI: Unconfirmed Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary - accounts for about 7.5 % of the total code. The helper could be + accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: - http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While - having this helper in bash rather than binary form certainly has an - impact on performance, it should be possible to minimize this impact + http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html + While having this helper in bash rather than binary form certainly has + an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 8 Title: Turn quilt into a noarch package - Hackweek VI: Unconfirmed + Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 9 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. + #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) + A noarch quilt package is now available at: + https://build.opensuse.org/package/show?package=quilt&project=home% + 3Ajdelvare%3AFactory -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 10 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory + #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) + The maintainers of the Debian quilt package have accepted all my + proposed bug fixes already: + http://git.debian.org/?p=collab-maint/quilt.git + (http://git.debian.org/?p=collab-maint/quilt.git) -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 11 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory + #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) + I've just published a new version with more performance improvements. + Expectedly, it's still somewhat slower than the original C + implementation, but it should be hardly noticeable in practice now. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 12 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) I've just published a new version with more performance improvements. Expectedly, it's still somewhat slower than the original C implementation, but it should be hardly noticeable in practice now. + #6: Jean Delvare (jdelvare) (2011-03-03 10:48:17) (reply to #3) + I've just published a new version with more performance improvements. + The restore and nolinks functions of backup-files are almost on-par + with the original implementation, performance-wise, so "quilt pop" is + almost as fast as it used to be. The only command which is still + somewhat slow is "quilt snapshot", I'll work on it next week. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 13 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) I've just published a new version with more performance improvements. Expectedly, it's still somewhat slower than the original C implementation, but it should be hardly noticeable in practice now. #6: Jean Delvare (jdelvare) (2011-03-03 10:48:17) (reply to #3) I've just published a new version with more performance improvements. The restore and nolinks functions of backup-files are almost on-par with the original implementation, performance-wise, so "quilt pop" is almost as fast as it used to be. The only command which is still somewhat slow is "quilt snapshot", I'll work on it next week. + #7: Jean Delvare (jdelvare) (2011-03-10 10:58:25) (reply to #3) + Yesterday I've updated the package in project home:jdelvare:Factory. + Performance is very good now, including for "quilt snapshot", and we + are very close to an upstream submission. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 14 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Developer: (Novell) - Developer: (Novell) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) I've just published a new version with more performance improvements. Expectedly, it's still somewhat slower than the original C implementation, but it should be hardly noticeable in practice now. #6: Jean Delvare (jdelvare) (2011-03-03 10:48:17) (reply to #3) I've just published a new version with more performance improvements. The restore and nolinks functions of backup-files are almost on-par with the original implementation, performance-wise, so "quilt pop" is almost as fast as it used to be. The only command which is still somewhat slow is "quilt snapshot", I'll work on it next week. #7: Jean Delvare (jdelvare) (2011-03-10 10:58:25) (reply to #3) Yesterday I've updated the package in project home:jdelvare:Factory. Performance is very good now, including for "quilt snapshot", and we are very close to an upstream submission. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 15 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) I've just published a new version with more performance improvements. Expectedly, it's still somewhat slower than the original C implementation, but it should be hardly noticeable in practice now. #6: Jean Delvare (jdelvare) (2011-03-03 10:48:17) (reply to #3) I've just published a new version with more performance improvements. The restore and nolinks functions of backup-files are almost on-par with the original implementation, performance-wise, so "quilt pop" is almost as fast as it used to be. The only command which is still somewhat slow is "quilt snapshot", I'll work on it next week. #7: Jean Delvare (jdelvare) (2011-03-10 10:58:25) (reply to #3) Yesterday I've updated the package in project home:jdelvare:Factory. Performance is very good now, including for "quilt snapshot", and we are very close to an upstream submission. + #8: Jean Delvare (jdelvare) (2012-01-31 11:46:14) (reply to #3) + I have rebased the package in project home:jdelvare:Factory on quilt + version 0.51 (from devel:tools:scm.) Upstream submission will happen + this week. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 16 Title: Turn quilt into a noarch package Hackweek VI: Implementation Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) I've just published a new version with more performance improvements. Expectedly, it's still somewhat slower than the original C implementation, but it should be hardly noticeable in practice now. #6: Jean Delvare (jdelvare) (2011-03-03 10:48:17) (reply to #3) I've just published a new version with more performance improvements. The restore and nolinks functions of backup-files are almost on-par with the original implementation, performance-wise, so "quilt pop" is almost as fast as it used to be. The only command which is still somewhat slow is "quilt snapshot", I'll work on it next week. #7: Jean Delvare (jdelvare) (2011-03-10 10:58:25) (reply to #3) Yesterday I've updated the package in project home:jdelvare:Factory. Performance is very good now, including for "quilt snapshot", and we are very close to an upstream submission. #8: Jean Delvare (jdelvare) (2012-01-31 11:46:14) (reply to #3) I have rebased the package in project home:jdelvare:Factory on quilt version 0.51 (from devel:tools:scm.) Upstream submission will happen this week. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) + #9: Jean Delvare (jdelvare) (2012-02-02 14:40:26) + All my work is upstream now, and should be released by the end of the + month. -- openSUSE Feature: https://features.opensuse.org/311072
Feature changed by: Jean Delvare (jdelvare) Feature #311072, revision 17 Title: Turn quilt into a noarch package - Hackweek VI: Implementation + Hackweek VI: Done Priority Requester: Desirable Requested by: Jean Delvare (jdelvare) Partner organization: openSUSE.org Description: Even though it is mainly made of bash and perl scripts, quilt is currently arch-specific. This is caused by one small helper binary names backup-files. The source code of this small helper binary accounts for about 6.7% of the total code. The helper could be rewritten in bash, and actually this has been done already. Debian reportedly uses a patch that does exactly this for 4 years now: http://www.mail-archive.com/quilt-dev@nongnu.org/msg01137.html While having this helper in bash rather than binary form certainly has an impact on performance, it should be possible to minimize this impact with some work. Also, virtually every piece of quilt could be rewritten in C for better performance, but performance was hardly the point of quilt in the first place. So, the goal of this hackweek project is to get rid of backup-files in its C form, and convert it to bash. Business case (Partner benefit): openSUSE.org: This change will improve the portability of quilt by limiting the build requirements. This change will improve the build time of quilt in OBS. This change will make it easier to contribute to quilt, by lowering the skills required. Discussion: #1: Klaas Freitag (kfreitag) (2011-01-18 13:39:39) I think this is a bug report rather than a hackweek project. #2: Jean Delvare (jdelvare) (2011-01-18 22:18:47) (reply to #1) I'm not quite sure why you say that. Quilt works as is, so we can't call it a bug. This is only a bizarre design to mix programming language when you really don't have to. #3: Jean Delvare (jdelvare) (2011-02-04 18:08:59) A noarch quilt package is now available at: https://build.opensuse.org/package/show?package=quilt&project=home% 3Ajdelvare%3AFactory #5: Jean Delvare (jdelvare) (2011-02-10 22:42:41) (reply to #3) I've just published a new version with more performance improvements. Expectedly, it's still somewhat slower than the original C implementation, but it should be hardly noticeable in practice now. #6: Jean Delvare (jdelvare) (2011-03-03 10:48:17) (reply to #3) I've just published a new version with more performance improvements. The restore and nolinks functions of backup-files are almost on-par with the original implementation, performance-wise, so "quilt pop" is almost as fast as it used to be. The only command which is still somewhat slow is "quilt snapshot", I'll work on it next week. #7: Jean Delvare (jdelvare) (2011-03-10 10:58:25) (reply to #3) Yesterday I've updated the package in project home:jdelvare:Factory. Performance is very good now, including for "quilt snapshot", and we are very close to an upstream submission. #8: Jean Delvare (jdelvare) (2012-01-31 11:46:14) (reply to #3) I have rebased the package in project home:jdelvare:Factory on quilt version 0.51 (from devel:tools:scm.) Upstream submission will happen this week. #4: Jean Delvare (jdelvare) (2011-02-06 21:06:49) The maintainers of the Debian quilt package have accepted all my proposed bug fixes already: http://git.debian.org/?p=collab-maint/quilt.git (http://git.debian.org/?p=collab-maint/quilt.git) #9: Jean Delvare (jdelvare) (2012-02-02 14:40:26) All my work is upstream now, and should be released by the end of the month. + #10: Jean Delvare (jdelvare) (2012-05-31 14:00:12) (reply to #9) + Quilt version 0.60 is packaged in both devel:tools:scm and openSUSE: + Factory now. First product with this feature will be openSUSE 12.2. -- openSUSE Feature: https://features.opensuse.org/311072
participants (1)
-
fate_noreply＠suse.de