Comments (12)
Actually make
also recompiles the binary.
I've never done it myself, but I believe the work-around is to touch a file when the build completes and have that file as a make dependency.
from nixnote2.
The first line in your quote is not a "compile step", but its is a "link step" (or "assembly of the binary").
But its true, that for some reason the "link step" is executed twice. I did not notice before.
Total build time with "clean" on my machine is 3 minutes.
Just the "2x link" on my machine takes 7 seconds, so eliminating the one duplicate link step would save ~3.5 seconds, which is about 2% of the build time.
So I don't think its critical, but I agree that doing the "link" twice is stupid and I will try to find out why this happens. Thx for reporting.
from nixnote2.
I marked it as "enhancement" as it is only ~2% of the build time and only affects the users, which want to compile themselves.
from nixnote2.
Awesome :) You've got a nice fast box, I envy you :-P
from nixnote2.
There is a lot of difference when are you using parallel processing: make
vs. e.g. make -j8
. Try the "-j" param, maybe it will be also faster on your box.
from nixnote2.
Nice one. I'm running with make -j $(nproc)
from nixnote2.
The solution to prevent calling "link" twice is just run "make install" not "make && make install".
So I fixed that in my build script and in the travis build script.
Thx for pointing that out.
So I close this. If you still have the problem - then pls comment here.
from nixnote2.
Frankly speaking that is weird -- a plain call of make should only build binaries and a call of "make install" should not try to build again if it has been built -- I'd say the problem is not solved but only circumvented in a dirty way.
from nixnote2.
I believe this is what we're after:
https://stackoverflow.com/questions/23371141/why-does-gnu-make-always-re-link-my-project
from nixnote2.
@hosiet:
😄 "dirty" is nickname of this project ...
The truth is, that with the time I have available for this project, if I would try to do it the "correct way" and first try to learn all the technologies needed, then I wouldn't even start coding... It touched "Makefiles" last time like 15 years ago and also that was briefly.
But you are right and you challenged me, so I looked it up and fixed the wrong dependency in the project. Now it works correctly. "make" creates the binaries, "make install" does the install and will not call link again.
(Its not that, I like the project being "dirty"... I try to improve, but... I try to be pragmatic and make it work, with what I have available).
from nixnote2.
.. so here is the latest Travis build log: https://api.travis-ci.com/v3/job/151309461/log.txt
it did "make && make install" and the "link" was executed once
from nixnote2.
The balance between doing it "correctly" and actually doing it is a a tough one. I probably err on the pedantic side and get less done as a result.
Brilliant - thank you :)
from nixnote2.
Related Issues (20)
- ubuntu22.04 SSL Errors HOT 2
- AppImage: "This doesn't look like a squashfs image." HOT 1
- Release page: invalid formatting (in one line) HOT 5
- 2.1.8: please update this link https://github.com/robert7/nixnote2/releases/download/continuous-develop/NixNote2-x86_64.AppImage HOT 4
- AppImage file name must contain version number HOT 5
- Shortcuts don't seem to be syncing HOT 2
- regression introduced in sorting code (pull #187) - narrow note list no longer sorted HOT 3
- Clarity on build pipeline, branches, tags HOT 3
- Releases: please provide binary for macOS
- regression since v2.1.9: nixnote2 no longer exiting cleanly (terminated by SIGABRT) HOT 14
- master branch triggers an unhandled exception in EDAMErrorCode::RATE_LIMIT_REACHED exception handling HOT 4
- develop branch - ccache added build dependency for the build-with-qmake.sh script not explicitely documented HOT 4
- debian packaging - wrong debian changelog package name
- Feature Request: add link to the images HOT 1
- Bug reseting background color of a note HOT 1
- Bug: Expanded icon and collapse icon don't change the visual HOT 1
- Need some help for building on Windows HOT 5
- Tags UI: not enough of space
- UI: dark theme: title in unreadable (white on white) when active (after mouse click)
- Windows: quick freezes (2-3 sec) each ~15secs HOT 2
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from nixnote2.