Ticket #3753 (closed task: fixed)
extfs: tester: let test scripts easily access configure-time parameters.
Reported by: | mooffie | Owned by: | mooffie |
---|---|---|---|
Priority: | major | Milestone: | 4.8.19 |
Component: | mc-vfs | Version: | master |
Keywords: | Cc: | ||
Blocked By: | #3752 | Blocking: | #3751 |
Branch state: | merged | Votes for changeset: | committed-master |
Description
This patch introduces a mechanism by which tests can easily access configure-time parameters, like @PERL@, @AWK@, etc.
(The traditional method, of *.in files, can still be used of course, but this mechanism is more comfortable.)
Rationale
As seen in #3751, tests should have access to configure-time parameters. After committing this patch, the file 'data/rpm.rewrite.sh' could be written as follows:
. "$MC_TEST_EXTFS_CONFIG_SH" blah_blah_blah() { $PERL -e 'whatever perl code' }
Attachments
Change History
Changed 8 years ago by mooffie
comment:2 Changed 8 years ago by zaytsev
- Status changed from new to accepted
- Owner set to zaytsev
- Milestone changed from Future Releases to 4.8.19
comment:3 follow-up: ↓ 4 Changed 8 years ago by zaytsev
I have convened with Andrew earlier this week, and we both agree that on the basis of your contributions you should be granted commit access to the source repository (a long time ago ;-) ). Today, I have cleaned up our GitHub organization settings and sent you an invitation to join the committers team. If anything, this will save us all a lot of time posting patches to Trac & creating branches from them. What do you think?
In terms of workflows, I believe that you might have followed our conversation with Egmont on the mailing list, and I hope that this also works for you... I'd be happy to discuss whatever you think is worth being discussed and help you if you need any help, but vacation time is over, and so I will be able to make as much time than I spent in the last couple of weeks :-/
comment:4 in reply to: ↑ 3 Changed 8 years ago by mooffie
Replying to zaytsev:
I have convened with Andrew earlier this week, and we both agree that [...] sent you an invitation to join the committers team
Andrew and Yury: thank you for your vote of confidence!
What do you think?
Superb.
In terms of workflows, I believe that you might have followed our conversation with Egmont on the mailing list,
Yes.
This post (and this one) contains everything.
I already started the workflow for #3752 (which blocks this ticket). Once I merge that one I'll handle this ticket.
(I won't be here till Sunday.)
comment:5 Changed 8 years ago by zaytsev
- Owner changed from zaytsev to mooffie
- Status changed from accepted to assigned
Awesome!
comment:6 Changed 8 years ago by mooffie
- Branch state changed from no branch to on review
branch: 3753_extfs_tester_access_configure_parameters
changeset:69897e94fd996ff92ad3df09bb82dae18add0320
comment:7 Changed 8 years ago by zaytsev
- Votes for changeset set to zaytsev
- Branch state changed from on review to approved
comment:8 Changed 8 years ago by mooffie
- Status changed from assigned to testing
- Votes for changeset changed from zaytsev to committed-master
- Resolution set to fixed
- Branch state changed from approved to merged
Merged to master: [b229abc1eb4f6bb204cf783f72a8d05bbbc0d367]