-
Notifications
You must be signed in to change notification settings - Fork 56
Support using the auto-detected name #355
base: master
Are you sure you want to change the base?
Conversation
|
You'll also need to modify Makefile.toplevel to:
...and add |
|
The |
00a7a22 to
28a8644
Compare
|
@phmccarty, @bryteise Could I have your review, please? |
b6fc152 to
114e815
Compare
114e815 to
716b2fe
Compare
716b2fe to
e4719d0
Compare
473718d to
5cbdfdb
Compare
Rework to absorb a lot of the work we were doing in makefiles so that we can avoid setting the package name manually in 'make autospecnew'.
When using github urls, the code tries to test the detected name against the previously provided name. If we didn't provide a name, though, this would throw an exception.
5cbdfdb to
35665e2
Compare
|
Remaining work, in case someone else has time to pick up the torch:
|
|
@bryteise Is this feature still in scope for autospec? I think it would be nice to autodetect the package name, but due to the extra complexity introduced with the interaction with the common tooling, we were somewhat undecided about what needed to change in that tooling, and autospec, or both, to accommodate... |
|
Hrm I'm a little unsure about all the bits that need to be reworked. Because |
|
I agree that all the name detection logic should be handled by autospec. The idea of adding a |
|
Edit: One second look, this PR adds the makefile write routine, so disregard. Edit 2: IIRC, at the time, I wasn't convinced that autospec writing the makefile was a good idea, since the primary user of it is the common tooling, not autospec. Perhaps autospec could write a separate makefile (say, |
Rework to absorb a lot of the work we were doing in makefiles so that we
can avoid setting the package name manually in 'make autospecnew'.