summaryrefslogtreecommitdiff
path: root/src
Commit message (Collapse)AuthorAgeFilesLines
...
* Bump version to 3.3.2Fred Hebert2016-10-141-1/+1
|
* Merge pull request #1337 from ferd/support-aliasingalisdair sullivan2016-10-131-14/+30
|\ | | | | Properly support package aliasing and alt names
| * Track package hash in memory index, add hash testFred Hebert2016-10-041-1/+17
| | | | | | | | | | | | | | | | | | | | This adds tracking of package hash in the in-memory index rather than the current `undefined' values. According to the test added, this is not necessary for transitive package dep hash chcking, but does result in a more complete index search result when doing app lookups, and could yield some optimizations on hash checks by checking from the index structure before fetching a package.
| * Properly support package aliasing and alt namesFred Hebert2016-09-271-13/+13
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Aliasing only had a bit of ad-hoc support in rebar3, and various issues have encountered problems related to the package names not mapping properly with the application name. One such issue is https://github.com/erlang/rebar3/issues/1290 The problem has been hard to find because it only impacts transitive dependencies (not top-level ones) of other packages. The root cause for this is that the application name was not being tracked by rebar3's internal index, only the package name and its version were. When a given application was a package app, the data for the application name would be reconstructed from the lock file, but only if it were a top-level app or a dependency of a source application where parsing the lock file is necessary to know what comes next. When a transitive dependency of a package dependency was fetched, we instead read its dependencies directly from the in-memory package index within rebar3. This caused us to only read the package name and version, and lost all information regarding application name. This worked fine for most cases since for the vast majority of packages, the package name matches the app name, but failed for all aliases, which would then be moved to directories that wouldn't match the app name. This in turn broke some aspects of code analysis (in Dialyzer), or other functionality relying on static paths, such as including .hrl files from dependencies. This patch reformats the internal storage format of dependencies to align with the internal one used by rebar3, so that the app name can be carried along with the package name and its version. The fix can only work once `rebar3 update` is called so the index is rebuilt internally, and will the file cached on disk will be incompatible with older rebar3 versions. Currently, the following is not covered: - Tests - Including the package hashes of dependencies so they may match what is in a lock file -- they're being `undefined` instead, which may break some lookups. The previous format did not lend itself to hashing in the same way, and it is possible transitive deps were not being tracked properly, or worked by respecting the current package hierarchy. This will require further analysis For now this commit can allow reviewing and discussion.
* | Merge pull request #1348 from ElectronicRU/masteralisdair sullivan2016-10-133-6/+24
|\ \ | | | | | | | | | | | | Fix sys.config's handling in common_test and otherwise. closes #1289
| * | Some post-review changes:Alexander Sedov2016-10-112-0/+6
| | | | | | | | | | | | | | | - restore path after loading applications, - helpful comments.
| * | Avoid backward-compatibility-breaking changes.Alexander Sedov2016-10-113-13/+15
| | |
| * | Made reading sys.configs consistent with OTP specification.Alexander Sedov2016-10-113-10/+13
| | |
| * | Rereading system configuration sets up persistent options if possible.Alexander Sedov2016-10-111-1/+7
| | |
| * | Made Common Test load the user's applications before slurping config.Alexander Sedov2016-10-111-0/+1
| | |
* | | Get stacktrace only once and as soon as possible. Fixes #1347.Alexander Sedov2016-10-131-2/+3
|/ / | | | | | | | | It turns out that pretty-printing uses throw-catch and thus garbages out the stack trace sometimes, so we should get it only once.
* | Use all_src_dirs for include pathsFred Hebert2016-09-301-2/+2
| | | | | | | | Helps cover extra cases.
* | Fix private includes when compiling in test profileFred Hebert2016-09-301-9/+14
| | | | | | | | | | | | | | | | | | | | | | | | | | | | When an include file is set in a private path (i.e. src/), the rebar3 compiler would not add them to the {i, Path} params -- only include/ and the project root were being added. This meant that when some extra source directories were added to the compile job, such as test/ when running under the test profile, the private include paths could not be shared with the test module. This patch fixes the issues (and adds tests) for such a specific case by adding all the configured include paths to the {i, Path} erl_opts arguments, yielding successful compile runs.
* | Add support for behaviors, and not just behavioursBrujo Benavides2016-09-282-1/+4
|/
* allow using an alternate regex to locate test modules during eunit runsalisdair sullivan2016-09-201-7/+12
| | | | | | | | {`eunit_test_regex`, Regex}` will use the supplied `Regex` instead of the default to locate tests in test dirs. note this matches only the filename, not the path. the regex is applied to all test dirs, recursively fixes #1331
* Merge pull request #1332 from erlang/rebar-1327Fred Hebert2016-09-201-1/+15
|\ | | | | compile + cover compile prior to analyzing coverdata
| * only compile/cover compile when generating analysisalisdair sullivan2016-09-201-4/+13
| | | | | | | | don't compile when resetting coverdata
| * cover compile prior to calculating coveragealisdair sullivan2016-09-201-1/+6
| | | | | | | | fixes #1327
* | recompile all files when a parse transform given as an opt needs updatingalisdair sullivan2016-09-201-1/+15
|/ | | | | | | | | there's no way to detect which files actually rely on a parse transform passed to the compiler via the options (as opposed to `-compile(..)` so if any parse transforms are in modules that need recompiling just recompile the world fixes #1328
* Merge pull request #1321 from vans163/error_on_load_file_failureFred Hebert2016-09-151-2/+13
|\ | | | | log error if a module failed to load_file, the user should be aware
| * replace error with debugvans1632016-09-031-2/+2
| |
| * log error if a module failed to load_file, the user should be awarevans1632016-09-031-2/+13
| |
* | Ignore mv warningsDavid de Boer2016-09-151-3/+8
| | | | | | | | | | | | | | In some cases, mv will throw a warning, while still moving the files correctly and returning a 0 return code: "mv: can't preserve ownership of ... Permission denied".
* | Back to semver version post releaseFred Hebert2016-09-021-1/+1
| |
* | Bump to 3.3.1Fred Hebert2016-09-021-1/+1
|/
* delete purge load_file specific ordervans1632016-09-021-1/+1
|
* Make less than R17 compatiblevans1632016-09-011-7/+11
|
* comment and spacing fixupvans1632016-09-011-5/+4
|
* Update rebar_agent.erlvans1632016-09-011-3/+3
| | | opps. Infinite loop fixed.
* Update rebar_agent.erlvans1632016-09-011-2/+29
| | | | https://github.com/erlang/rebar3/pull/1317 In reference to with support to load erlang code atomically but load nifs non-atomically.
* Merge pull request #1318 from tuncer/conventional-stacktraceFred Hebert2016-08-311-1/+1
|\ | | | | Print stacktrace in a more conventional way
| * Print stacktrace in a more conventional wayTuncer Ayaz2016-08-301-1/+1
| | | | | | | | | | | | Insert a newline before printing the stacktrace so that the term is easier to read and copy. This is a more conventional way to print traces, and is, for instance, the way it's done by make and python.
* | reset accumulated coverdata on writing out to disk. this preventsalisdair sullivan2016-08-271-0/+2
| | | | | | | | | | provider chains like `eunit, ct, proper` from misreporting cover stats from providers later in the sequence
* | Merge pull request #1315 from ferd/fix-hashcheck-noindexFred Hebert2016-08-271-1/+10
|\ \ | | | | | | Fix crash when doing hash check with missing index
| * | Fix crash when doing hash check with missing indexFred Hebert2016-08-271-1/+10
| |/ | | | | | | | | | | | | | | | | | | Specifically, when fetching an application where the expected hash is unknown, the hash is validated from the hex index; when the index is available, the hash is fetched fine and later inserted in the lock file. However, if the index is not available, the call would simply crash. This patch fixes thing so that instead, the index is refreshed before giving up and failing.
* | Re-format cover exclusion codeFred Hebert2016-08-271-17/+26
| | | | | | | | | | | | - brings back former error handling and debug messages - keeps the filtering of excluded mods and debug messages - breaks up code into multiple functions and removes nesting
* | Merge branch 'cover_excl_mods' of https://github.com/lpgauth/rebar3 into ↵Fred Hebert2016-08-271-15/+23
|\ \ | |/ |/| | | lpgauth-cover_excl_mods
| * Add support for cover_excl_modsLouis-Philippe Gauthier2016-06-151-15/+23
| |
| * Cutting 3.1.1Fred Hebert2016-05-171-1/+1
| |
* | Returning to git-based taggingFred Hebert2016-08-261-1/+1
| |
* | Bumping to 3.3.0Fred Hebert2016-08-261-1/+1
| |
* | Equivalent trim_all in bin split for <17.xFred Hebert2016-08-251-1/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | The trim_all option used in binary:split/3 is not supported in 17.x. This patch makes an equivalent operation by eliminating empty split fragments. From the docs: trim Removes trailing empty parts of the result (as does trim in re:split/3. trim_all Removes all empty parts of the result. The new expression is therefore equivalent to the old one, but with the added benefit of compatibility. Fixes #1275
* | Make Edoc carry paths of pre-built librariesFred Hebert2016-08-251-9/+26
| | | | | | | | | | | | | | | | | | | | | | Given the topological sort applied to top-level apps, we should be able to carry the edoc values for paths configured when more than one app exists. This allows multiple disjoint app to have defined cross-linking in the documentation. Tests pending.
* | Print error on too many help argumentsRasmus Svensson2016-08-151-1/+4
| | | | | | | | | | | | | | Previously the help task would crash when given more than two arguments. After this change it instead print a message: Too many arguments given. Usage: rebar3 help [<namespace>] <task>
* | Make the escriptize provider hookableFred Hebert2016-08-131-2/+7
| | | | | | | | This will allow to move and modify the generated files
* | Merge pull request #1298 from ferd/escript-filter-paths-fixFred Hebert2016-08-101-1/+3
|\ \ | | | | | | Fix filtering of system libraries in escriptize
| * | Fix filtering of system libraries in escriptizeFred Hebert2016-08-101-1/+3
| | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | https://github.com/erlang/rebar3/pull/1249 introduced a mechanism by which escript dependencies of applications only would be included; this required adding a filter to skip system libraries in the OTP root because that tends to break escripts in very nasty ways. However, the problem came that some libraries are just not in the escript path but may still be included; for these libraries the path prefix check failed as they return `{error, bad_name}` from `code:lib_dir(Dep)` rather than just the path they're in -- specifically, this happens with top level apps. The issue was reported in https://github.com/erlang/rebar3/issues/1294 and the current patch fixes it by accepting a `bad_name` dep as valid, since it is obviously not in the root path.
* | | Merge pull request #1288 from ElectronicRU/masterFred Hebert2016-08-101-1/+1
|\ \ \ | | | | | | | | Typo fix.
| * | | Typo fix.Alexander Sedov2016-08-041-1/+1
| |/ /
* | | Merge pull request #1282 from ssbx/custom_template_dirFred Hebert2016-08-101-1/+1
|\ \ \ | | | | | | | | template_dir option was forgotten in documentation plus recurse in di…