| Commit message (Collapse) | Author | Age | Files | Lines |
... | |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
This simplifies git_project_patch_recursive() and provides a unified
method for patching, regardless of the patch format (mailbox or
diff).
Logic is included for dealing with patching failures due to
git_apply_check() not being enough to catch certain edge cases, such
as 'git apply --check' returning 0 but a subsequent 'git apply'
failing.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Since changes from an applied diff need to be added to the index and
committed (vs. mailbox files with the git_patch function) new
functions were necessary to account for this difference.
This also fixes a bug where diffs were applied but not committed,
at minimum dirtying the work tree and potentially causing problems for
any later patches.
|
|/
|
|
|
|
| |
The previous function, diff_patch_file, was more error-prone and
fragile and did not take into consideration patching of more than one
file at a time(!).
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Patching before removing blobs is cleaner since patches are applied
against a freshly created and checked out branch at the specified
revision. Applying patches first also seems to reduce the likelihood
of failed patching attempts due to a nebulous error about files not
matching the git index (but doesn't eliminate it). More debugging
required.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Previously the only way patching would work is if for every target
there existed a corresponding directory under $project/$PATCHES, e.g:
"./libreboot update coreboot x200 8MiB corebootfb grub"
would require the "grub" directory and its parents:
"projects/coreboot/patches/x200/8MiB/corebootfb/grub"
Now you only need target-specific patch directories if you need them;
I believe this was the intended behavior.
Superfluous quotes were removed for readability.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
There was a bug where only $project was used as the branch name due to
some faulty logic which checked for the existence of a revision file
before setting the variables "prepare_branch" and "prepare_path".
The bug in this case was that prepare_branch would be set to the value
of $branch in only the first iteration of the for loop since in almost
all cases there is only one revision file that exists for a given
project, regardless of the arguments provided. Explained another way,
in order for the proper branch name to be used a revision file would
have had to exist in every target directory.
This was an issue because only one branch was ever operated on (named
$project), meaning if actions were performed on many targets then only
the last to run would be represented in the project's
repository--making tracking down some bugs a bit harder.
With this fixed we now create a branch for every possible project
or tool configuration and leaves us with a log for each.
|
| |
| |
| |
| |
| |
| |
| | |
This is a slight mitigation against branch name collisions. More
robust handling should be added in the future but this is simple
enough and provides some clear indication which branches have been
created by the build system.
|
|/
|
|
|
| |
A patch failing to apply shouldn't leave the repository in such a
state where manual intervention is necessary to reset it.
|
|\ |
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The 16MiB ROM is non-functional at the moment so an 8MiB target has
been added in order to avoid losing QEMU support while the bug is
identified and fixed.
Files were copied from the 16MiB directory with the exception of the
config file located at 8MiB/config which is new.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
The config for text mode doesn't actually have to be there (since
text mode is the default in the base config) but, as in other cases,
this configuration snippet is included for the sake of
documenting what part of the base config is related to the
target ("textmode" in this case).
|
| |
| |
| |
| |
| | |
16MiB is the largest flash size for most of our supported boards so
any emulated boards should stick with that upper limit.
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
The 16MiB ROM is non-functional at the moment so an 8MiB target has
been added in order to avoid losing QEMU support while the bug is
identified and fixed.
Files were copied from the 16MiB directory with the exception of the
config file located at 8MiB/config which is new.
|
| | |
|
| |
| |
| |
| |
| |
| |
| |
| | |
The config for text mode doesn't actually have to be there (since
text mode is the default in the base config) but, as in other cases,
this configuration snippet is included for the sake of
documenting what part of the base config is related to the
target ("textmode" in this case).
|
| |
| |
| |
| |
| | |
16MiB is the largest flash size for most of our supported boards so
any emulated boards should stick with that upper limit.
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
Updated with the "oldconfig" make target.
New configurations:
* USE_OPTION_TABLE=y
* STATIC_OPTION_TABLE=y
* COMPRESS_SECONDARY_PAYLOAD=y
|
|/
|
|
|
|
|
|
| |
Updated with the "oldconfig" make target.
New configurations:
* COMPRESS_SECONDARY_PAYLOAD=y
|
|\ |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|\ \ |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| | | |
|
| |/ |
|
|\ \
| |/
|/| |
|
| | |
|
| | |
|
| | |
|
| | |
|
| | |
|
|/ |
|
|\ |
|
| |
| |
| |
| | |
Those actions are only required when using separate home/boot volumes.
|
|\ \ |
|
| | |
| | |
| | |
| | | |
Output is fixed with a5babe37f42cc162812f8ed3310a1e0c45eabb67
|
| | |
| | |
| | |
| | | |
Makes it easier to read when multiple terse actions are performed.
|
|\ \ \
| |/ /
|/| | |
|
| | |
| | |
| | |
| | |
| | |
| | | |
Avoid redundant actions if certain projects which are part of
another (e.g. crossgcc in the coreboot repository) are left out of the
download process.
|
|\ \ \ |
|
|/ / / |
|