patches.tar.gz
|
Duplicate |
|||
| is duplicated by |
|
Resolve confusion over custom e2fsprogs | Resolved |
[LU-929] add configure --with-ldiskfsprogs to e2fsprogs Created: 15/Dec/11 Updated: 27/Jun/14 |
|
| Status: | In Progress |
| Project: | Lustre |
| Component/s: | None |
| Affects Version/s: | Lustre 2.1.0 |
| Fix Version/s: | None |
| Type: | Improvement | Priority: | Minor |
| Reporter: | Ramiro Alba Queipo | Assignee: | Jian Yu |
| Resolution: | Unresolved | Votes: | 0 |
| Labels: | e2fsprogs, llnl | ||
| Attachments: |
patches.tar.gz
|
||||||||
| Issue Links: |
|
||||||||
| Epic: | e2fsprogs | ||||||||
| Rank (Obsolete): | 10295 | ||||||||
| Description |
|
This is an adaptation of the patches for bug: https://bugzilla.lustre.org/show_bug.cgi?id=22983 Enables a new configure option (--with-ldiskfsprogs) which combined with the same option in lustre, allows e2fsprogs essential commands to be installed without conflicting e2fsprogs package on distribution |
| Comments |
| Comment by Andreas Dilger [ 20/Jan/12 ] |
|
Hi Ramiro, Patches for e2fsprogs should be submitted via Gerrit for testing and inspection. Please see the wiki pages: http://wiki.whamcloud.com/display/PUB/Submitting+Changes e2fsprogs changes are tracked using the tools/e2fsprogs git repository. |
| Comment by Peter Jones [ 05/Apr/12 ] |
|
Chris Is this something that LLNL would like to push upstream? If so, could you please upload the changes into gerrit with the appropriate signoff? Thanks Peter |
| Comment by Christopher Morrone [ 05/Apr/12 ] |
|
I am willing to do a push, but I will not have time to iterate on it at the moment. It certainly would be nice to stop carrying it ourselves. The notable problem would be that the patch doesn't completely convert the support libraries to ldiskfs naming and package those libraries. If one attempts a dynamic build of ldiskfsprogs, the binaries will wind up linking dynamically with e2fsprog's helper library and break in strange ways. So, yes, I'll post if that is helpful. Our changes are public here if you want a preview: |
| Comment by Christopher Morrone [ 19/Apr/12 ] |
|
I rebased our two main ldiskfsprogs patches onto WC's latest and submitted them: http://review.whamcloud.com/2582 |
| Comment by Peter Jones [ 19/Apr/12 ] |
|
YuJian Could you please assess these patches? Thanks Peter |
| Comment by Christopher Morrone [ 27/Jun/12 ] |
|
Updated patches for master-lustre, now that Whamcloud has dropped the use of quilt. http://review.whamcloud.com/3204 |
| Comment by Jian Yu [ 02/Jul/12 ] |
|
Hello Chris, Thanks for the patches. I just reviewed http://review.whamcloud.com/3204. Besides some build issues, I found there were still plenty of "e2fsck", "debugfs", etc. commands which were not substituted with "@FSCKPROG@", "@DEBUGFSPROG@", etc. in many files (like *.8.in and *.spec.in files). I'm wondering whether we need make all of these changes in those files. Do you have any thoughts on this? |
| Comment by Christopher Morrone [ 02/Jul/12 ] |
|
It would be a good idea to make those changes yes. I've been porting forward the patches for quite some time, but not actively seeking out new changes that could use those substitutions. |